Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=666311 --- Comment #4 from Ed Marshall <esm+redhat@xxxxxxxxx> 2011-02-15 12:03:49 EST --- Great feedback, thank you! - The strip is a remnant from Chitlesh's first stab at packaging this; I've pulled that out. - Good catch on the bundled libraries; I have it building now without the included boost, quazip, and qtlockedfile. I did a sanity check across the repository history to make sure there aren't local changes that haven't made it upstream; there's a few small changes, but they all appear to be related to incorporating the libraries locally. I may have found a bug in the packaging for quazip as a result of this. :) (Boost is an interesting situation; the resulting binary doesn't actually end up linking against any of the supplied libraries, as it appears everything they're using from it inlines at compile-time. It's even more interesting, in that boost is called out as an example in No_Bundled_Libraries. :)) - I took a look at "ff", and the original upstream appears to be at http://shygypsy.com/tools/ . It's all separately-released files (effectively public domain/do-what-you-want, from the license at the top of the page) rather than collected into a library, and the fellow doesn't appear to be actively maintaining it. It also looks like they're patching it locally for Fritzing, making them effectively upstream for this. - The CXXFLAGS export has been added. - I've also updated to this morning's bugfix release of 1.5.1b to address the autorouter crashes with off-board parts. I'll post an update as soon as the mock builds finish. Thanks again for the help. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review