Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"edkrepo sync" support for tags #134

Open
ndhaller opened this issue Feb 25, 2023 · 0 comments
Open

"edkrepo sync" support for tags #134

ndhaller opened this issue Feb 25, 2023 · 0 comments
Assignees
@ashedesimone

Comments

@ndhaller
Copy link
Contributor

edkrepo manifest file Combination and PatchSet can include git tag information, but "edkrepo sync" does not detect or notify user if tags locally and tags on the Remote have somehow deviated from each other.

In some other cases, "edkrepo sync" and "edkrepo sync --fetch" may not create all the same local tags as a fresh 'edkrepo clone' will.

Manually running 'git fetch --tags' on each of the repo seems to work as a workaround for both cases.

edkrepo 3.1.0, git version 2.39.1.windows.1
@ashedesimone ashedesimone self-assigned this Feb 27, 2023
ashedesimone added a commit to ashedesimone/edk2-edkrepo that referenced this issue Mar 17, 2023
@ashedesimone
EdkRepo: Add --tags flag to sync command
1d41b6f 
Add support for fetching tags.

Additionally move messages indicating that the sync
is starting for a repo to the begining of that process
to improve messaging.

Fixes tianocore#134

Signed-off-by: Ashley E Desimone <[email protected]>
@ashedesimone ashedesimone mentioned this issue Mar 17, 2023
Closed
ashedesimone added a commit to ashedesimone/edk2-edkrepo that referenced this issue Apr 10, 2023
@ashedesimone
EdkRepo: Add --tags flag to sync command
fb41245 
Add support for fetching tags.

Additionally move messages indicating that the sync
is starting for a repo to the begining of that process
to improve messaging.

Fixes tianocore#134

Signed-off-by: Ashley E Desimone <[email protected]>
ashedesimone added a commit to ashedesimone/edk2-edkrepo that referenced this issue Apr 10, 2023
@ashedesimone
EdkRepo: Add --tags flag to sync command
075b5a2 
Add support for fetching tags.

Additionally move messages indicating that the sync
is starting for a repo to the begining of that process
to improve messaging.

Fixes tianocore#134

Signed-off-by: Ashley E Desimone <[email protected]>
ashedesimone added a commit to ashedesimone/edk2-edkrepo that referenced this issue Apr 10, 2023
@ashedesimone
EdkRepo: Add --tags flag to sync command
0dce084 
Add support for fetching tags.

Additionally move messages indicating that the sync
is starting for a repo to the begining of that process
to improve messaging.

Fixes tianocore#134

Signed-off-by: Ashley E Desimone <[email protected]>
kevinsun49 pushed a commit to ashedesimone/edk2-edkrepo that referenced this issue Apr 11, 2023
@ashedesimone @kevinsun49
EdkRepo: Add --tags flag to sync command
a5b859b 
Add support for fetching tags.

Additionally move messages indicating that the sync
is starting for a repo to the begining of that process
to improve messaging.

Fixes tianocore#134

Signed-off-by: Ashley E Desimone <[email protected]>
Reviewed-by: Kevin Sun <[email protected]>
ashedesimone added a commit to ashedesimone/edk2-edkrepo that referenced this issue Apr 11, 2023
@ashedesimone
EdkRepo: Add --tags flag to sync command
03b0028 
Add support for fetching tags.

Additionally move messages indicating that the sync
is starting for a repo to the begining of that process
to improve messaging.

Fixes tianocore#134

Signed-off-by: Ashley E Desimone <[email protected]>
Reviewed-by: Kevin Sun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ashedesimone ashedesimone

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

EdkRepo: Add --tags flag to sync command ashedesimone/edk2-edkrepo
2 participants
@ndhaller @ashedesimone