Page MenuHomeFreeBSD

databases/grass7: Unbreak on 11i386
ClosedPublic

Authored by lbartoletti on Jul 18 2020, 8:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 25, 10:01 AM
Unknown Object (File)
Sun, Sep 21, 6:07 AM
Unknown Object (File)
Mon, Sep 15, 10:22 PM
Unknown Object (File)
Sep 6 2025, 11:22 PM
Unknown Object (File)
Aug 30 2025, 1:49 AM
Unknown Object (File)
Aug 22 2025, 1:24 AM
Unknown Object (File)
Aug 18 2025, 11:00 PM
Unknown Object (File)
Aug 10 2025, 7:20 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 32469
Build 29945: 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.