Skip to content
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

doebuild.py: PVR includes -r0 if explicitly specified #1028

Closed
wants to merge 1 commit into from

Conversation

ulm
Copy link
Member

@ulm ulm commented Apr 29, 2023

PMS says that PVR is "Package version and revision (if any)". https://projects.gentoo.org/pms/8/pms.html#x1-109001r1

Bug: https://bugs.gentoo.org/875362

@thesamesam
Copy link
Member

Would you mind mentioning the paludis+pkgcore research in the commit message, as well as the commit you dug up originally adding this?

Thank you for this!

PMS says that PVR is "Package version and revision (if any)".
https://projects.gentoo.org/pms/8/pms.html#x1-109001r1

Pkgcore and Paludis follow PMS, i.e. PVR includes an explicit -r0
(in other words, ${PF} is always equal to ${PN}-${PVR} for them).

The previous logic for PVR had been added in 2001:
https://gitweb.gentoo.org/archive/repo/gentoo-2.git/commit/?id=9aa2cf18b4de3c81fd5bc98718b2614bd2e73d23

Bug: https://bugs.gentoo.org/875362
Closes: gentoo#1028
Signed-off-by: Ulrich Müller <[email protected]>
@ulm
Copy link
Member Author

ulm commented Apr 29, 2023

Would you mind mentioning the paludis+pkgcore research in the commit message, as well as the commit you dug up originally adding this?

Commit message updated. (No change of the code.)

@thesamesam
Copy link
Member

Thanks!

@ulm ulm deleted the pvr branch April 29, 2023 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants