Bug 221316
| Summary: | math/dynare: Add missing dependencies | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Fernando Apesteguía <fernape> | ||||
| Component: | Individual Port(s) | Assignee: | Kurt Jaeger <pi> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Some People | CC: | pi, riggs | ||||
| Priority: | --- | Keywords: | patch-ready | ||||
| Version: | Latest | Flags: | pi:
merge-quarterly+
|
||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| See Also: | https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221187 | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 221342 | ||||||
| Attachments: |
|
||||||
|
Description
Fernando Apesteguía
2017-08-07 16:57:49 UTC
Nice clear report Fernando thank you. Don't forget to add the second issue (version update) to this issues 'Blocks' field, so that this one gets committed first. Also, you may be able to use LIBS= in the port Makefile instead of patching configure (In reply to Kubilay Kocak from comment #1) Will do. I was waiting to see if this gets committer as is to make the new patch on top of this one. I'll have a look at the configure patch and the use if LIBS. Thanks! Done. Can we move forward with this? testbuilds@work (In reply to Kurt Jaeger from comment #4) Are you going to look at bug 221342 afterwards? If not, I can take a look as well. yes, I'll look at 221342 when the testbuilds are fine and this change is committed. A commit references this bug: Author: pi Date: Sat Aug 12 17:21:43 UTC 2017 New revision: 447843 URL: https://svnweb.freebsd.org/changeset/ports/447843 Log: math/dynare: Add missing dependencies - Regenerated patches with "make makepatch" to avoid portlint warnings - Added LICENSE - Added missing library dependencies PR: 221316 Submitted by: Fernando Apesteguia <fernando.apesteguia@gmail.com> (maintainer) Changes: head/math/dynare/Makefile head/math/dynare/files/patch-Makefile.in head/math/dynare/files/patch-configure head/math/dynare/files/patch-journal.cweb head/math/dynare/files/patch-parser-exception.h A commit references this bug: Author: pi Date: Sat Aug 12 17:22:56 UTC 2017 New revision: 447844 URL: https://svnweb.freebsd.org/changeset/ports/447844 Log: math/dynare: added patches I missed in last commit PR: 221316 Submitted by: Fernando Apesteguia <fernando.apesteguia@gmail.com> (maintainer) Changes: head/math/dynare/Makefile head/math/dynare/files/patch-dynare++_kord_journal.cweb head/math/dynare/files/patch-dynare++_parser_cc_parser__exception.h head/math/dynare/files/patch-mex_build_octave_configure A commit references this bug: Author: pi Date: Sat Aug 12 18:12:39 UTC 2017 New revision: 447846 URL: https://svnweb.freebsd.org/changeset/ports/447846 Log: MFH: r447843 r447844 math/dynare: Add missing dependencies - Regenerated patches with "make makepatch" to avoid portlint warnings - Added LICENSE - Added missing library dependencies PR: 221316 Submitted by: Fernando Apesteguia <fernando.apesteguia@gmail.com> (maintainer) Approved by: portmgr (fix dependencies blanket) Changes: _U branches/2017Q3/ branches/2017Q3/math/dynare/Makefile branches/2017Q3/math/dynare/files/patch-Makefile.in branches/2017Q3/math/dynare/files/patch-configure branches/2017Q3/math/dynare/files/patch-dynare++_kord_journal.cweb branches/2017Q3/math/dynare/files/patch-dynare++_parser_cc_parser__exception.h branches/2017Q3/math/dynare/files/patch-journal.cweb branches/2017Q3/math/dynare/files/patch-mex_build_octave_configure branches/2017Q3/math/dynare/files/patch-parser-exception.h Committed, thanks! |