Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
3 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
05583ab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@michaelld This commit (on my machine) breaks the installation of the gr-fcdproplus port with:
Reverting to the previous version of the portfile (with 1.1.1) fixes the compile issue. I haven't tracked down if this is a log4cpp issue or a gr-fcdproplus, but wanted to give everyone a heads-up since gr-fcdproplus is a dependency of gr-osmosdr.
05583ab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see this issue too. Strangely, not all OOT modules exhibit it. I have a few ideas that I'll be checking out.
05583ab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gr-fcdproplus doesn't even reference log4cpp ... so, I'm guessing this is a related to moving to using "-defined dynamic_lookup" so-as to not link directly to a specific Python library (in core GNU Radio libraries). Any OOT either needs to link to liblog4cpp or use the -undefined flag. I'm investigating further.
05583ab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed here: https://trac.macports.org/changeset/91dd35333bbcbd39eeaabf17ba037eeed20d9304/macports-ports