databases/grass7: Unbreak on 11i386

[lbartoletti]

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.

[lbartoletti]

Bump portrevision

[lbartoletti]

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

[tcberner]

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

145 ↗

(On Diff #74609)

can't you use LLVM_DEFAULT?

[lbartoletti]

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.

[tcberner]

databases/grass7/Makefile

144 ↗

(On Diff #74778)

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

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

[lbartoletti]

Since LLVM_DEFAULT is 90 we can use it.

[lbartoletti]

databases/grass7/Makefile

144 ↗

(On Diff #74778)

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

[tcberner]

databases/grass7/Makefile

144 ↗

(On Diff #74833)

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

[lbartoletti]

remove LLVM_VER and use ${LLVM_DEFAULT}

[lbartoletti]

databases/grass7/Makefile

144 ↗

(On Diff #74833)

erg... of course :)

[tcberner]

ship it :)