Page MenuHomeFreeBSD
Differential D14719

Add missing LIB_DEPENDS as reported in stage-qa
ClosedPublic
Actions

Authored by fernape on Mar 17 2018, 12:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Aug 19, 9:44 PM
Unknown Object (File)
Mon, Aug 11, 5:13 PM
Unknown Object (File)
Thu, Jul 31, 11:43 AM
Unknown Object (File)
Jul 21 2025, 10:48 AM
Unknown Object (File)
Jul 17 2025, 1:41 PM
Unknown Object (File)
Jul 7 2025, 12:40 PM
Unknown Object (File)
Jul 7 2025, 10:30 AM
Unknown Object (File)
Jul 5 2025, 6:46 PM
View All Files
Subscribers
mat

Details

Reviewers
tz
tcberner
Commits
rP465112: math/dynare: Add missing LIB_DEPENDS to fix stage-qa
Summary

Add missing LIB_DEPENDS as reported in stage-qa

Test Plan

cd math/dynare && portlint -AC && make
It should have a clean output now.

Poudriere logs for {10.3,10.4,11.1}{i386,amd64} and 12i386
https://www.dropbox.com/sh/z0581x5p6opj7vd/AAAUDcCWKUU_5c_hoiki9gGaa?dl=0

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 15626
Build 15662: arc lint + arc unit

Event Timeline

fernape created this revision.Mar 17 2018, 12:01 PM
Herald added a subscriber: mat. · View Herald TranscriptMar 17 2018, 12:01 PM
Harbormaster completed remote builds in B15615: Diff 40379.Mar 17 2018, 12:01 PM
fernape edited the test plan for this revision. (Show Details)Mar 17 2018, 12:31 PM
tcberner added inline comments.Mar 17 2018, 4:38 PM
math/dynare/Makefile
24

There's no need to add it twice.

tcberner added a comment.Mar 17 2018, 4:40 PM

You have an OCTAVE Option already, so is ovtave always required?

fernape updated this revision to Diff 40396.Mar 17 2018, 11:51 PM

Add OCTAVE_LIB_DEPENDS dependency.

Poudriere builds for {10.3,10.4,11.1}{amd64,i386} and 12 i386
available https://www.dropbox.com/sh/nf49c231hdwi08e/AABQ87bkb3ko1U9KplQSXK9aa?dl=0

Harbormaster completed remote builds in B15626: Diff 40396.Mar 17 2018, 11:52 PM
fernape updated this revision to Diff 40397.Mar 17 2018, 11:54 PM

I uploaded by mistake some other files I was working on...

tcberner added inline comments.Mar 18 2018, 6:39 AM
math/dynare/Makefile
22

^ trailing \

39

^ you can get rid of the BUILD_DEPENDS as you add the LIB_DEPENDS.

fernape updated this revision to Diff 40411.Mar 18 2018, 4:21 PM

Address both problems.

New logs in https://www.dropbox.com/sh/nws7okkqi69lqg8/AABDld-J-2U6qav1za1zlN00a?dl=0

Harbormaster completed remote builds in B15631: Diff 40411.Mar 18 2018, 4:21 PM
tcberner added inline comments.Mar 18 2018, 9:15 PM
math/dynare/Makefile
38–39

^no need for +=

fernape updated this revision to Diff 40441.Mar 19 2018, 4:43 PM

Replace '+=' with '=' in OCTAVE_LIB_DEPENDS

Harbormaster completed remote builds in B15647: Diff 40441.Mar 19 2018, 4:43 PM
tz accepted this revision.Mar 20 2018, 10:19 AM

Looks fine to me

This revision is now accepted and ready to land.Mar 20 2018, 10:19 AM
Closed by commit rP465112: math/dynare: Add missing LIB_DEPENDS to fix stage-qa (authored by fernape). · Explain WhyMar 20 2018, 10:26 PM
This revision was automatically updated to reflect the committed changes.
fernape marked 4 inline comments as done.

Revision Contents
Changeset List

PathSize
cad/
openvsp/
10 lines
devel/
shellcheck/
39 lines
3 lines
12 lines
math/
dynare/
5 lines

Diff 40396

View Options

cad/openvsp/Makefile

Loading...
View Options

devel/shellcheck/

Loading...
View Options

devel/shellcheck/Makefile

Loading...
View Options

devel/shellcheck/distinfo

Loading...
View Options

devel/shellcheck/pkg-descr

Loading...
View Options

math/dynare/Makefile

Loading...