Page MenuHomeFreeBSD

emulators/virtualbox-ose{,-legacy}: sort out options
ClosedPublic

Authored by vvd on Jun 24 2023, 6:58 PM.
Tags
Referenced Files
Unknown Object (File)
Sat, Nov 16, 8:54 AM
Unknown Object (File)
Tue, Nov 12, 5:54 AM
Unknown Object (File)
Tue, Nov 12, 5:31 AM
Unknown Object (File)
Tue, Nov 12, 5:06 AM
Unknown Object (File)
Tue, Nov 12, 5:00 AM
Unknown Object (File)
Sat, Nov 9, 6:28 PM
Unknown Object (File)
Sat, Nov 9, 5:24 PM
Unknown Object (File)
Fri, Nov 8, 7:01 AM

Details

Summary

Sort out options: order (portlint), convert to OPTION_*: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=200764
Added entry in "Troubleshooting" about "VERR_NO_MEMORY": https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=252227
Force devel/llvm15 for emulators/virtualbox-ose-legacy on recent 14 with llvm16 in base.

Tested build in poudriere.
And on live system 13.2 amd64 build and run both ports.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

vvd requested review of this revision.Jun 24 2023, 6:58 PM
vvd created this revision.

Tested build:
emulators/virtualbox-ose: poudriere and64 12.4, 13.1, 13.2; live 13.2.
emulators/virtualbox-ose-legacy: poudriere and64 and i386 12.4, 13.1, 13.2.

emulators/virtualbox-ose-legacy/pkg-message
86

Change to:

the wiki

87

Change to:

https

(Some of these are comments because Phabricator seems to prevent suggestions for this review.)

emulators/virtualbox-ose-legacy/pkg-message
74
emulators/virtualbox-ose/files/pkg-message.in
74
88
vvd edited the summary of this revision. (Show Details)
vvd marked 5 inline comments as done.
emulators/virtualbox-ose-legacy/Makefile
157

I think it won't hurt to define these vars unconditionally.

162

Same here.

emulators/virtualbox-ose-legacy/Makefile
157

It won't break the following lines if DOCS off?

post-patch:
…
        @${ECHO_CMD} 'VBOX_PATH_DOCBOOK = ${DBKXSLDIR}' >> \
            ${WRKSRC}/LocalConfig.kmk
        @${ECHO_CMD} 'VBOX_PATH_DOCBOOK_DTD = ${DBKXMLDIR}' >> \
            ${WRKSRC}/LocalConfig.kmk
162

Nothing broken anywhere?

Update patch with git diff -U10000.

emulators/virtualbox-ose-legacy/Makefile
157

Hmmm, who knows. Then to be on the safe side:

DOCS_VARS= vbox_path_docbook=${DBKXSLDIR}

See https://docs.freebsd.org/en/books/porters-handbook/book/#options-vars

emulators/virtualbox-ose-legacy/Makefile
157

Do this trick work for:

PYDISTUTILS_PKGNAME=    vboxapi
PYDISTUTILS_PKGVERSION= 1.0

too?

PYTHON_VARS=            pydistutils_pkgname=vboxapi pydistutils_pkgversion=1.0
emulators/virtualbox-ose-legacy/Makefile
157

Yep it should.

vvd edited the summary of this revision. (Show Details)
vvd marked 4 inline comments as done.

That's a good cleanup, thanks!

This revision is now accepted and ready to land.Jun 28 2023, 6:15 AM