Created attachment 194217 [details] grass 7.4.0_3 Hi, Some users and I noticed errors with the package: poudriere testport seemed OK, but bulk or install return pkg-static errors. It seems there are pkg-plist errors. So I updated the port to include some minor changes and remove lines in pkg-plist that are "incorrect". BLAS are also a difficulty, since GOTO and OPEN BLAS have not the same pkg-plist with NETLIB (default) and ATLAS conflicts with numpy (use NETLIB as default). I need to deepen the subject, in the meantime I propose to use only NETLIB. I also noticed some difficulties during the run, I added a pkg message. I would like an exp-run before merging this patch to be sure that now, is not broken. Thank you.
I do not understand, there is exactly one port that depend on this one, graphics/qgis, so, what would you expect requesting an exp-run to achieve exactly? Build 32k packages for nothing and then build the 2 affected? As you state that you already ran poudriere testport, an exp-run will give you even less informations, as it will not run in testing mode.
(In reply to Mathieu Arnold from comment #1) No, there's no exp-build required, I think it can be ignored
I can sneak the patch in another exp-run
You're right, I forgot that an exp-run had to recompile everything, sorry. My request was mainly linked to the fact that there are differences between a "poudriere testport" and a "poudriere bulk" or "make install".
It fails during patch phase: =======================<phase: patch >============================ ===> Patching for grass7-7.4.0_3 ===> Applying FreeBSD patches for grass7-7.4.0_3 sed: -I or -i may not be used with stdin
Created attachment 194243 [details] patch-v2 This version builds in poudriere. Can someone run-test it ? It has small changes to fix the sed-issue and some pkg-plist fixes.
Sorry for the delay, Seems ok for me. What did you think about adding a pkg-message for this comment: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228605#c4 ? Thanks
Created attachment 196775 [details] patch-to-7.4.1 Update to 7.4.1 with pkg-message
testbuilds are fine, but please note the build problem if build via qgis, see PR#228605
A commit references this bug: Author: pi Date: Thu Nov 29 08:39:01 UTC 2018 New revision: 486155 URL: https://svnweb.freebsd.org/changeset/ports/486155 Log: databases/grass7: update 7.4.0 -> 7.4.1 PR: 228975 Submitted by: lbartoletti@tuxfamily.org (maintainer) Relnotes: https://trac.osgeo.org/grass/wiki/Release/7.4.1-News Changes: head/databases/grass7/Makefile head/databases/grass7/distinfo head/databases/grass7/pkg-message head/databases/grass7/pkg-plist
Committed, thanks.
(In reply to Kurt Jaeger from comment #11) Hi Kurt, Thanks for the commit. Two issues should be mentioned: - It seems you forgot to remove the following two lines in databases/grass7/Makefile of version 7.4.1 DEPRECATED= Broken for more than 6 months EXPIRATION_DATE= 2018-12-19 - GRASS versions have to be hardcoded in some dependend ports, e.g. graphics/qgis. It would be nice if you could also change the following line in graphics/qgis/Makefile GRASS_CMAKE_ON= -DWITH_GRASS7:BOOL=TRUE -DGRASS_PREFIX7=${LOCALBASE}/grass-7.4.0 Many thanks again, Rainer
thanks for the pointer. I'm not sure 7.4.1 as is works (the previous was broken as well, so...). I'll testbuild with 7.4.3, then test for qgis as well. If those all work, the DEPRECATE can be removed.
(In reply to Rainer Hurling from comment #12) > GRASS_CMAKE_ON= -DWITH_GRASS7:BOOL=TRUE -DGRASS_PREFIX7=${LOCALBASE}/grass-7.4.0 This can be done differently (especially in version 3 of QGIS), I'll prepare a patch for you in the week