Page MenuHomeFreeBSD

databases/grass7: Unbreak on 11i386
ClosedPublic

Authored by lbartoletti on Jul 18 2020, 8:34 AM.
Tags
None
Referenced Files
F81627727: D25712.diff
Fri, Apr 19, 5:25 AM
Unknown Object (File)
Thu, Mar 21, 4:29 PM
Unknown Object (File)
Feb 18 2024, 1:33 PM
Unknown Object (File)
Feb 18 2024, 1:32 PM
Unknown Object (File)
Feb 18 2024, 1:32 PM
Unknown Object (File)
Feb 18 2024, 1:32 PM
Unknown Object (File)
Feb 18 2024, 1:32 PM
Unknown Object (File)
Feb 18 2024, 1:32 PM
Subscribers

Details

Summary

When I added the option for compiling with openmp, like other ports, I don't use compiler:openmp but directly the default compiler because it works just as well. However, on FreeBSD 11, grass only builds well with LLVM90.

While I'm here I added with-proj-[includes|libs] to be more thorough (There's a problem on 13i386 that I still can't solve properly) and I remove useless post-patch commented.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 32503
Build 29976: arc lint + arc unit

Event Timeline

BTW this commit could be merged quarterly to unbreak databases/grass7 11i386 on quarterly too.

Is there really a need to bump revision? -- if the package failed to build then you can just leave it as is. If the package was built but broken, then yes, the bump is required :)

databases/grass7/Makefile
144

can't you use LLVM_DEFAULT?

Remove useless PORTREVISION. My bad, I had planned to make other small changes but I preferred to delete them and forgot to remove this tag.

Use LLVM_DEFAULT instead of LLVM_VER, but CC/CPP/etc. are always necessary since just LLVM_DEFAULT and BUILD_DEPENDS are not enough.

databases/grass7/Makefile
144

^ I meant to use the value of ${LLVM_DEFAULT} by the framework :) and not set it manually

=> see Mk/bsd.default-versions.mk

Since LLVM_DEFAULT is 90 we can use it.

lbartoletti added inline comments.
databases/grass7/Makefile
144

My bad, I misread the value of LLVM_DEFAULT, for me it still was < 90 ... :)

databases/grass7/Makefile
144

^ you could just use ${LLVM_DEFAULT} everywhere without defining your own LLVM_VER :)

lbartoletti marked an inline comment as done.

remove LLVM_VER and use ${LLVM_DEFAULT}

lbartoletti added inline comments.
databases/grass7/Makefile
144

erg... of course :)

This revision is now accepted and ready to land.Aug 20 2020, 10:28 AM
This revision was automatically updated to reflect the committed changes.
lbartoletti marked an inline comment as done.