-
Notifications
You must be signed in to change notification settings - Fork 5
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
120 jar files are not present in the ubuntu debian khiops desktop packages #121
120 jar files are not present in the ubuntu debian khiops desktop packages #121
Conversation
280f451
to
0a29212
Compare
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.
I'd squash all the commits related to the packaging.
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.
Can't we make the same trick for as in build-linux-pack-containers.yml
for GH_REPOSITORY
?
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.
I don't see how, because in that other case, I had added a preliminary stage doing this. Whereas here, we would need this variable from the outset (before any job stage is executed).
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.
Ok, let's leave it like that for the moment.
I would too, but I wanted to make sure we agree on this before squashing. |
GitHub workflow syntax does not provide a means of inlining string manipulations. Hence: - when needed in a job step, a step has been added for generating environment variable which lowercases the path to the repository - when needed in a job specification, the path has been reverted to its hard-coded lowercased version.
0a29212
to
586fc94
Compare
closes #120