-
Notifications
You must be signed in to change notification settings - Fork 507
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.
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
[msbuild] Remove usage of GitInfo in Xamarin.Messaging.Build. #19935
Conversation
* The calculation GitInfo does is horribly wrong for us - it seems to pick a tag the most recent tag and parse it to pick out a version number: * This means for any given release it'll use the previous tag - because we don't necessarily tag until after the release build has been committed. * It's not a stable version: a build is made (finding one tag), a new tag is pushed, and if the same commit is built, it may find the new tag, thus creating a new version number. * We may have multiple tags for any given commit. * The version calculation is also wrong: right now it finds the tag `xamarin-ios-15.5.0.5-4407-gd45f4e90a6`, and comes up with the version number 0.5.0.4407. * I didn't investigate deeply, but GitInfo also touches .git/index and .git/HEAD somehow, which causes us to rebuild stuff. * We already have an established way of calculating version numbers and other build related information, so use that after extending it a bit.
…led in. This is required to avoid GitInfo until these two PRs have been merged and flowed to us: * xamarin/XamarinVS#14408 * xamarin/Xamarin.Messaging#700 See also xamarin#19935.
💻 [CI Build] Tests on macOS M1 - Mac Ventura (13.0) passed 💻✅ All tests on macOS M1 - Mac Ventura (13.0) passed. Pipeline on Agent |
💻 [CI Build] Tests on macOS M1 - Mac Big Sur (11.5) passed 💻✅ All tests on macOS M1 - Mac Big Sur (11.5) passed. Pipeline on Agent |
📚 [PR Build] Artifacts 📚Packages generatedView packagesPipeline on Agent |
💻 [CI Build] Windows Integration Tests passed 💻✅ All Windows Integration Tests passed. Pipeline on Agent |
✅ API diff for current PR / commitLegacy Xamarin (No breaking changes)
NET (empty diffs)
✅ API diff vs stableLegacy Xamarin (No breaking changes).NET (No breaking changes)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
🚀 [CI Build] Test results 🚀Test results✅ All tests passed on VSTS: simulator tests. 🎉 All 166 tests passed 🎉 Tests counts✅ cecil: All 1 tests passed. Html Report (VSDrops) Download Pipeline on Agent |
…led in. (#19937) This is required to avoid GitInfo until these two PRs have been merged and flowed to us: * xamarin/XamarinVS#14408 * xamarin/Xamarin.Messaging#700 See also #19935.
tag the most recent tag and parse it to pick out a version number:
don't necessarily tag until after the release build has been committed.
is pushed, and if the same commit is built, it may find the new tag,
thus creating a new version number.
xamarin-ios-15.5.0.5-4407-gd45f4e90a6
, and comes up with the versionnumber 0.5.0.4407.
.git/HEAD somehow, which causes us to rebuild stuff.
build related information, so use that after extending it a bit.