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
Linuxulator: ports for Rocky Linux 9 #255
Conversation
@TijlCoosemans can you look at this? |
I haven't looked at everything in detail, but here are some general comments:
|
Mk/Uses/linux.mk
Outdated
&& empty(NOT_FOR_ARCHS:Mamd64) && ${linux_ARGS} != rl9 | ||
DISTFILES_amd64?= ${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_i386}:amd64,i386/} \ | ||
${LIB_DISTNAMES_i386:S/$/${EXTRACT_SUFX_i386}:amd64,i386/} \ | ||
${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | ||
${LIB_DISTNAMES_amd64:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | ||
${BIN_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | ||
${SHARE_DISTNAMES:S/$/${EXTRACT_SUFX_noarch}/} | ||
. endif | ||
. if !(defined(ONLY_FOR_ARCHS) && empty(ONLY_FOR_ARCHS:Mamd64)) \ | ||
&& empty(NOT_FOR_ARCHS:Mamd64) && ${linux_ARGS} == rl9 | ||
DISTFILES_amd64?= ${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | ||
${LIB_DISTNAMES_amd64:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | ||
${BIN_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | ||
${SHARE_DISTNAMES:S/$/${EXTRACT_SUFX_noarch}/} | ||
. endif |
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.
&& empty(NOT_FOR_ARCHS:Mamd64) && ${linux_ARGS} != rl9 | |
DISTFILES_amd64?= ${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_i386}:amd64,i386/} \ | |
${LIB_DISTNAMES_i386:S/$/${EXTRACT_SUFX_i386}:amd64,i386/} \ | |
${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${LIB_DISTNAMES_amd64:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${BIN_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${SHARE_DISTNAMES:S/$/${EXTRACT_SUFX_noarch}/} | |
. endif | |
. if !(defined(ONLY_FOR_ARCHS) && empty(ONLY_FOR_ARCHS:Mamd64)) \ | |
&& empty(NOT_FOR_ARCHS:Mamd64) && ${linux_ARGS} == rl9 | |
DISTFILES_amd64?= ${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${LIB_DISTNAMES_amd64:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${BIN_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${SHARE_DISTNAMES:S/$/${EXTRACT_SUFX_noarch}/} | |
. endif | |
&& empty(NOT_FOR_ARCHS:Mamd64) | |
. ifndef DISTFILES_amd64 | |
. if ${linux_ARGS} == c7 | |
DISTFILES_amd64= ${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_i386}:amd64,i386/} \ | |
${LIB_DISTNAMES_i386:S/$/${EXTRACT_SUFX_i386}:amd64,i386/} | |
. endif | |
DISTFILES_amd64+= ${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${LIB_DISTNAMES_amd64:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${BIN_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ | |
${SHARE_DISTNAMES:S/$/${EXTRACT_SUFX_noarch}/} | |
. endif | |
. endif |
devel/linux_libusb/Makefile
Outdated
.if ${ARCH} == amd64 && ${FLAVOR:Mc7} | ||
ONLY_FOR_ARCHS+= i386 |
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.
ONLY_FOR_ARCHS isn't just for amd64. Also, is it customary to use :M with FLAVOR? I think you can just use ==.
.if ${ARCH} == amd64 && ${FLAVOR:Mc7} | |
ONLY_FOR_ARCHS+= i386 | |
.if ${FLAVOR} == c7 | |
ONLY_FOR_ARCHS+= i386 | |
.endif | |
.if ${ARCH} == amd64 && ${FLAVOR} == c7 |
95622c8
to
fc74e5d
Compare
Right, I will go over all the ports again and add plists for aarch64.
Fixed.
Let me explain. These ports do not install libraries but executables, so they can't be depended on via I see no sense in having two ports for mostly the same |
Ports like linux-wps-office and linux-chrome have a separate upstream who tend to make sure their packages work on multiple linux distributions while rl9 strace and wget only work on rl9. In that case it's easier to have separate ports for c7 and rl9 strace than to try to support both from one port. You can add them to linux.mk and depend on them using USE_LINUX. USE_LINUX adds RUN_DEPENDS by default. |
Sigh, you're right. That was silly of me to remove rl9 spice from these ports. |
570d02a
to
d426942
Compare
@TijlCoosemans I have applied your suggestions and also added aarch64 pkg-plists, carefully comparing each of them to the amd64 ones. I also addressed comments from the Bugzilla PR. Is this ready to go in? |
@@ -40,7 +46,8 @@ MAKE_ENV+= LDADD="-lrt" | |||
|
|||
MAKE_ARGS+= NO_WDATE_TIME=yes | |||
|
|||
.if ${ARCH} == amd64 | |||
.if ${FLAVOR} == c7 | |||
ONLY_FOR_ARCHS+= i386 | |||
PLIST_FILES+= usr/lib/libusb.so.3 \ |
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 PLIST_FILES must be behind .if ${ARCH} == amd64. It corresponds to the amd64 bits in do-build and do-install below
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.
Right, fixed.
Mk/Uses/linux.mk
Outdated
. endif | ||
. if !(defined(ONLY_FOR_ARCHS) && empty(ONLY_FOR_ARCHS:Mamd64)) \ | ||
&& empty(NOT_FOR_ARCHS:Mamd64) && ${linux_ARGS} == rl9 | ||
DISTFILES_amd64?= ${LIB_DISTNAMES:S/$/${EXTRACT_SUFX_amd64}:amd64/} \ |
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 .if block with DISTFILES_amd64 assignment should be removed. It is handled by the .if block above.
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.
Sorry, I just blindly copied your suggested changes. Fixed.
Sponsored by: Serenity Cybersecurity, LLC Co-authored-by: Dima Panov <fluffy@FreeBSD.org>
Sponsored by: Serenity Cybersecurity, LLC
Co-authored-by: Gleb Popov <arrowd@FreeBSD.org> Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Co-authored-by: Gleb Popov <arrowd@FreeBSD.org> Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC Co-authored-by: awoonya <mzhyneva@gmail.com>
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Sponsored by: Serenity Cybersecurity, LLC
Okay for me now. Do you know what the .buildid files are for? They are the only difference between pkg-plist.aarch64 and pkg-plist.amd64. |
and
I thought about not installing these at all, but that is orthogonal for this PR. |
Hello,
|
@arrowd