-
Notifications
You must be signed in to change notification settings - Fork 441
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
project_loader: allow loading null parts #2153
project_loader: allow loading null parts #2153
Conversation
This allows one to specify remote parts without requiring the use of `after`, which implies a relationship which may not exist. LP: #1772027 Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>
Codecov Report
@@ Coverage Diff @@
## master #2153 +/- ##
==========================================
+ Coverage 91.3% 91.31% +<.01%
==========================================
Files 193 193
Lines 12165 12167 +2
Branches 1812 1813 +1
==========================================
+ Hits 11107 11110 +3
Misses 715 715
+ Partials 343 342 -1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested this one locally with multiple remote parts, works great.
Since Snapcraft 2.43 it is possible to include a remote part without specify it in a part's `after` value. This allows parts that not actually depend on each other from being bond together by dependency , for example build step of yad part being cleaned due to change in desktop-gtk3. Refer-to: Bug #1772027 “Pull stage of an unspecified part is unexpectedly cleaned by snapcraft” : Bugs : Snapcraft <https://bugs.launchpad.net/snapcraft/+bug/1772027> Refer-to: project_loader: allow loading null parts by kyrofa · Pull Request #2153 · snapcore/snapcraft <canonical/snapcraft#2153> Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
Prefer this new syntax introduced in Snapcraft 2.43 to avoid the cleaning of the poedit part due to the modification of the desktop-gtk3 part. Refer: canonical/snapcraft#2153
Since Snapcraft 2.43 it's now possible to specify building a part without listing it in an existing part's after clause, which may cause the depending part unnecessarily cleaned due to the dependency. Refer canonical/snapcraft#2153 for more info about the syntax. Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
Prefer this new syntax introduced in Snapcraft 2.43 to avoid the cleaning of the poedit part due to the modification of the desktop-gnome-part part. Refer: canonical/snapcraft#2153
./runtests.sh static
?./runtests.sh unit
?This PR fixes LP: #1772027 by allowing one to specify remote parts without requiring the use of
after
, which implies a relationship which may not exist. With this PR, one can specify that the want a remote part simply by listing the part name, e.g.: