-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Move conf and comps to qubes-release #113
Conversation
8ddcdbb
to
35ba126
Compare
self.comps_path = self.comps_path.relative_to( | ||
self.get_sources_dir() / "qubes-release" |
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.
This doesn't work, I think relative_to
does something else. Should be maybe:
self.comps_path = self.comps_path.relative_to( | |
self.get_sources_dir() / "qubes-release" | |
self.comps_path = self.get_sources_dir() / "qubes-release" / self.comps_path |
?
(and similar for kickstart)
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.
And also, later usage in update_parameters
and in run
might need a change too. It tries to copy_in into installer/comps (which might be okay?), but it will be problematic for kickstart that do include other files (that are not copied in anymore)
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.
For the include part, to be honest right now, I've no idea. If we need to resolve all the includes that's going to be fun
e994e92
to
6540ec1
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #113 +/- ##
==========================================
+ Coverage 77.21% 77.40% +0.18%
==========================================
Files 46 46
Lines 5004 5001 -3
==========================================
+ Hits 3864 3871 +7
+ Misses 1140 1130 -10 ☔ View full report in Codecov by Sentry. |
This fails with qubes executor:
|
I set:
|
QubesOS/qubes-issues#9254