-
Notifications
You must be signed in to change notification settings - Fork 13
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
fix make obs-workdir target #25
Conversation
…references other than git tags
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.
is the print(f
a feature or typo?
@stefanotorresi thx. I think the printf format can be converted with
|
@MalloZup I used f-strings throughout the whole script! I don't see any compelling technical reason to change that now, to be honest. In fact, since the f-strings were introduced in Python, the best practice is to use those instead of |
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. thx @stefanotorresi . Sarcastic mode oN: I guess python is not made for entreprise software.
I think I never discovered in my life a language that during is lifetyime has changed so many time useless syntax for printing
This PR makes a few changes on how the
make obs-workdir
target works.It now uses the
HEAD
reference as a defaultVERSION
environment variable, and it now allows to also change theREPOSITORY
variable to make the OBStar_scm
source service fetch the sources from a different repository than the main upstream.Making OBS commits for Git feature branches on OBS branch packages is now documented in the development notes.
fixes #17