Bug 196577 - cad/elmerfem: Update to 8.1-20160311
Summary: cad/elmerfem: Update to 8.1-20160311
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Thierry Thomas
URL:
Keywords: needs-qa, patch
Depends on:
Blocks:
 
Reported: 2015-01-06 22:41 UTC by Pedro F. Giffuni
Modified: 2016-03-17 18:25 UTC (History)
4 users (show)

See Also:
koobs: maintainer-feedback-


Attachments
elmerfem update (4.57 KB, patch)
2015-01-06 22:41 UTC, Pedro F. Giffuni
no flags Details | Diff
elmerfem update 20150305 (5.63 KB, patch)
2015-03-05 21:25 UTC, Pedro F. Giffuni
no flags Details | Diff
Update to 7.0.20150430 (4.80 KB, patch)
2015-05-02 02:24 UTC, Pedro F. Giffuni
no flags Details | Diff
Update to version 8.0 (4.82 KB, patch)
2015-05-08 18:48 UTC, Pedro F. Giffuni
no flags Details | Diff
math/arpack-ng - use blaslapack (1.23 KB, patch)
2015-06-16 21:42 UTC, Pedro F. Giffuni
no flags Details | Diff
elmerfem - Update to 8.0.20150616 (5.12 KB, patch)
2015-06-16 21:43 UTC, Pedro F. Giffuni
no flags Details | Diff
elmerfem - Update to 8.0.20150625 (5.69 KB, patch)
2015-07-01 16:41 UTC, Pedro F. Giffuni
no flags Details | Diff
elmerfem - Update to 8.0.20150625 (5.51 KB, patch)
2015-07-01 18:41 UTC, Pedro F. Giffuni
no flags Details | Diff
elmerfem - Update to 8.0.20150818 (8.80 KB, patch)
2015-08-22 19:01 UTC, Pedro F. Giffuni
no flags Details | Diff
elmerfem - Update to 8.0.20150903 (8.80 KB, patch)
2015-09-09 21:35 UTC, Pedro F. Giffuni
no flags Details | Diff
elmerfem - Update to 8.0.20151231 (9.51 KB, patch)
2016-01-06 02:34 UTC, Pedro F. Giffuni
no flags Details | Diff
Update to version 8.1-20160311 (from yesterday) (9.88 KB, patch)
2016-03-12 17:45 UTC, Pedro F. Giffuni
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pedro F. Giffuni freebsd_committer freebsd_triage 2015-01-06 22:41:33 UTC
Created attachment 151433 [details]
elmerfem update

Update with changes up to 20141218. including a minor change to the build.

Initial try to have this use the blaslapack framework. Unfortunately it looks like the netlib blas is hardcoded in the CMake buildsystem so set it to netlib for now.

The blaslapack support involved touch also the arpack-ng port.
Comment 1 Thierry Thomas freebsd_committer freebsd_triage 2015-01-24 16:06:15 UTC
The blaslapack support involved touch also the arpack-ng port.
Comment 2 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-03-05 21:25:12 UTC
Created attachment 153869 [details]
elmerfem update 20150305

Update to latest version (arpack-ng remains untouched wrt previous patch)
Comment 3 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-05-02 02:24:03 UTC
Created attachment 156227 [details]
Update to 7.0.20150430

ping?
Comment 4 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-05-08 18:48:56 UTC
Created attachment 156517 [details]
Update to version 8.0
Comment 5 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-05-08 18:49:54 UTC
For reference:
pfg@gorilon:/usr/ports/cad/elmerfem % make check-plist
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
===> No pkg-plist issues found (check-plist)
Comment 6 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-06-16 21:42:03 UTC
Created attachment 157814 [details]
math/arpack-ng - use blaslapack

Split the arpack-ng patch - use blaslapack
Comment 7 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-06-16 21:43:57 UTC
Created attachment 157815 [details]
elmerfem - Update to 8.0.20150616
Comment 8 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-07-01 16:01:18 UTC
ping?
Comment 9 Kubilay Kocak freebsd_committer freebsd_triage 2015-07-01 16:20:25 UTC
This is the downside of a single issue for two ports (and separate maintainers).

Feedback/Maintainer timeout (4months) on math/arpack-ng change (while assigned to maintainer).

Reporter is committer, assign accordingly.

QA results (poudriere) would be handy.
Comment 10 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-07-01 16:41:47 UTC
Created attachment 158228 [details]
elmerfem - Update to 8.0.20150625

Update to 8.0.20150625.
Fix Bug 201246
Comment 11 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-07-01 16:42:36 UTC
Reporter not a ports committer: back to the pool.
Comment 12 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-07-01 18:41:11 UTC
Created attachment 158232 [details]
elmerfem - Update to 8.0.20150625

Cleaner fix for the LDCONFIG issue.
Comment 13 Kubilay Kocak freebsd_committer freebsd_triage 2015-07-05 16:55:48 UTC
(In reply to Pedro F. Giffuni from comment #11)

I'm happy to review this for you on Phabricator as I have for a few other not (yet) ports committers. Just add me as a reviewer so we can track it there and reference it here
Comment 14 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-07-05 19:02:30 UTC
(In reply to Kubilay Kocak from comment #13)

Thank you for the kind offer.

With due respect though, what is wrong with patches in bugzilla? I don't really have the setup for phabricator/ports, plus I like to keep the ports tree read-only so I don't have to type passphrases for updates (or accidentally commit something).
Comment 15 Kubilay Kocak freebsd_committer freebsd_triage 2015-07-07 05:42:10 UTC
(In reply to Pedro F. Giffuni from comment #14)

Nothing is wrong with patches in Bugzilla. Phabricator is the place where reviews can/should/ought to take place, which can save time going back and forth on bugzilla, especially if changes are required. 

I also think its important to have a documented review too, especially in the cases where the committer is not (yet) a ports committer.

Having said that, it's not a requirement, I was just offering :)
Comment 16 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-08-22 19:01:40 UTC
Created attachment 160223 [details]
elmerfem - Update to 8.0.20150818

Update and appease portlint.
Comment 17 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-09-09 21:35:49 UTC
Created attachment 160880 [details]
elmerfem - Update to 8.0.20150903

pfg@gorilon:/usr/ports/cad/elmerfem % make check-plist
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
===> No pkg-plist issues found (check-plist)
Comment 18 Martin Wilke freebsd_committer freebsd_triage 2015-12-05 09:28:12 UTC
Take.
Comment 19 Pedro F. Giffuni freebsd_committer freebsd_triage 2016-01-06 02:34:03 UTC
Created attachment 165133 [details]
elmerfem - Update to 8.0.20151231

Update to the latest snapshot.
Comment 20 Pedro F. Giffuni freebsd_committer freebsd_triage 2016-03-12 17:45:24 UTC
Created attachment 168047 [details]
Update to version 8.1-20160311 (from yesterday)


% make check-plist
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
===> No pkg-plist issues found (check-plist)

I will now step down as maintainer: after almost two years
trying to update this, the effort seems useless.
Comment 21 Kubilay Kocak freebsd_committer freebsd_triage 2016-03-13 11:15:10 UTC
@Pedro, is the issue the delay getting this change committed? If so, we should re-assign this bug (assignee timeout) so someone else other than Martin can commit it.

In fact, there is nothing precluding you from committing it except that there is not yet complete (poudriere) confirmation of it passing QA. check-plist is good but not entirely sufficient

Let me know how you want to proceed
Comment 22 Pedro F. Giffuni freebsd_committer freebsd_triage 2016-03-13 15:27:33 UTC
Hi Kubilay;

(In reply to Kubilay Kocak from comment #21)

I have never really gotten feedback about what may be failing/missing. Perhaps it was just the issue of the missing use of blaslapack in arpack-ng (which is not a a showstopper but still should be done).

I don't blame Martin, it just seems there is very little interest among committers to get this fixed/updated so I prefer to spend my time somewhere else.

And no, I don't have spare resources to set up poudriere.
Comment 23 Martin Wilke freebsd_committer freebsd_triage 2016-03-14 02:20:59 UTC
Sorry for the delay i better but it back to freebsd-ports-bugs.
Comment 24 Martin Wilke freebsd_committer freebsd_triage 2016-03-14 02:21:20 UTC
back to pool
Comment 25 Thierry Thomas freebsd_committer freebsd_triage 2016-03-17 18:25:07 UTC
Committed, thanks!
Comment 26 commit-hook freebsd_committer freebsd_triage 2016-03-17 18:25:31 UTC
A commit references this bug:

Author: thierry
Date: Thu Mar 17 18:24:39 UTC 2016
New revision: 411297
URL: https://svnweb.freebsd.org/changeset/ports/411297

Log:
  Update to version 8.1-20160311.
  Fix PR ports/201246.

  PR:		ports/196577
  Submitted by:	pfg

Changes:
  head/cad/elmerfem/Makefile
  head/cad/elmerfem/distinfo
  head/cad/elmerfem/files/patch-CMakeLists.txt
  head/cad/elmerfem/files/patch-elmergrid_src_CMakeLists.txt
  head/cad/elmerfem/files/patch-fem_src_CMakeLists.txt
  head/cad/elmerfem/files/patch-matc_src_CMakeLists.txt
  head/cad/elmerfem/files/patch-post-src-elmerpost.h
  head/cad/elmerfem/files/patch-post-src-screensave.c
  head/cad/elmerfem/pkg-plist