[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | --- Comment #21 from Peter Lemenkov lemen...@gmail.com 2009-12-24 14:32:14 EDT --- Unblocking FE-NEEDSPONSOR since I just sponsored Matt. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 matt chan talc...@gmail.com changed: What|Removed |Added Depends on||550234 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #20 from matt chan talc...@gmail.com 2009-12-24 00:06:00 EDT --- A quick update on our progress so far: The TNT (and JAMA) libraries have been abstracted out and packaged. The review requests are at 549980 and 550234. If someone has a second, could they review them quickly please? They are just a bunch of headers and have less than 25 files each package. It shouldn't take more than 30 mins for each. And on the tkhtml3 front, the former upstream dev has confirmed directly that the project is dead. The brlcad team have made provisions to take over the upstream for the STEP and Utah projects. The transition to tkPng will be made in the next release, and tkImg will be dropped as a requirement. Tkhtml3 and OpenNURBS are the remaining issues which have not been resolved yet. Matt -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 matt chan talc...@gmail.com changed: What|Removed |Added Depends on||549980 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #17 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-12-16 10:42:51 EDT --- When an upstream is dead like you're saying tkImg and tkHTML3 are, the problem of bundled and static libraries is exacerbated. In those cases, instead of having to wait for multiple upstreams to discover problems, make fixes, announce them, and then have the next upstream in the chain realise the problem affects their bundled libraries, make fixes, and release updated tarballs, we have upstreams whose source will never change even though there's known security vulnerabilities. This makes it even more imperative that the packager fixes these problems as soon as possible as the packager is the new upstream for the package and if they package with these problems then the maintenance burden for fixing those types of security problems falls entirely on them. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #18 from Christopher Sean Morrison brl...@mac.com 2009-12-16 15:48:46 EDT --- It's worth noting a few updates since this recent set of updates about tkImg and tkHTML3. First off, some clarifications. The tkImg package only bundles those external dependencies for download convenience and can be disabled. In fact, our bundling of tkImg itself in BRL-CAD was a simple subset of just the PNG Tcl bindings (without libpng, libz, or any other lib). That said .. we're already in the process of replacing tkImg with tkPNG since it's even more simple and is closer to the minimal functionality that we need. To top it off, we found existing RPMs for tkImg around the same time.. :) As for tkHTML3 and the assertion that things were being statically linked in, that was a mistake. The tkHTML3 sources don't even have any external dependencies, much less linking in anything static. There was some confusion inferred from a misleading statement on the website about a related code. So the basic summary, it's mostly all moot. We'll have to get tkPNG packaged, but that should be very easy. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #19 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-12-16 17:56:37 EDT --- The tkImg package only bundles those external dependencies for download convenience and can be disabled. Excellent. As long as those are disabled in the Fedora build it's perfectly acceptable :-) The rest of your update sounds very encouraging as well. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #14 from matt chan talc...@gmail.com 2009-12-16 00:29:36 EDT --- While creating a package for tkImg, I noticed that they include modified source versions of libz, libpng, libungif, libjpeg, and libtiff (from Aug 2000) so they can be loaded into the tcl/tk core. Does anyone care to weigh in on this? Does this count as a violation of Fedora's pre-packaged libraries clause? TkImg upstream seems to be dead or close to it. The last release was in Dec 2002 and there's only blips of activity on their sourceforge tracker so it may be difficult to have changes implemented. Matt -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #15 from Kevin Fenzi ke...@tummy.com 2009-12-16 00:35:03 EDT --- Yes. See: https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#When_a_Bundled_Library_is_Discovered_Post-Review Basically: file a bug on it, and make it block the blocker used to track these issues. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #16 from matt chan talc...@gmail.com 2009-12-16 00:49:32 EDT --- Another bit of fun while exploring tkHTML3 this time: Everything is statically linked in, from 2006 or so. I'm not sure what it would take to refactor to use dynamic libs, never tried before and I don't know if the lib versions from 2006 are still around. Upstream appears to be dead as well: http://groups.google.com/group/tkhtml3/browse_thread/thread/3eeb094b7b460e3a As before, any thoughts? Matt -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #6 from Chitlesh GOORAH chitl...@gmail.com 2009-10-06 08:08:41 EDT --- Ping any progress on this, Matt ? -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #7 from matt chan talc...@gmail.com 2009-10-06 10:32:19 EDT --- Ack. I'm sorry, I don't know how, but I missed the email about Kevin's comment. I guess there's a downside to too many bugzilla emails. I will have some time to work on this in the weekend to fix the license and rpmlint errors. I thought they were only trivial ones, but I didn't test the most recent changes. The libraries are somewhat of a sticky issue. I consulted with the BRL-CAD devs on the possibility of abstracting out the libraries while building this package. It appears that they have made heavy 3rd party modifications to most of these libraries, especially Utah, Template Numerical, openNURBS, and NIST STEP, and the upstream projects are unwilling to accept them or are no longer active. TkHTML is a dead project as far as I know. None of those libraries listed should be present in fedora 11. Is it still considered a good idea to abstract out the libraries, or should we just consider them a part of the BRL-CAD package? According to the BRL-CAD devs, they don't really resemble the original libraries/projects anymore. To my knowledge, there is no project outside of BRL-CAD that makes use of their modifications to these libs. Thoughts? Matt -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #8 from Kevin Fenzi ke...@tummy.com 2009-10-06 11:51:13 EDT --- Well, the guideline is at: https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries I really wish projects wouldn't do this kind of thing. It makes it much harder to get them packaged for a distribution. ;( -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #9 from matt chan talc...@gmail.com 2009-10-06 12:08:52 EDT --- Hmm that's the first time I've read that page. So what do we do now? Would applying to FeSCo be appropriate? This lib problem is also the reason other distributions are having trouble accepting BRL-CAD. (The lib naming problem being the other one. BRL-CAD's been in development for 20 years, so it doens't follow standard naming schemes.) Matt -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 Toshio Ernie Kuratomi a.bad...@gmail.com changed: What|Removed |Added CC||a.bad...@gmail.com --- Comment #10 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-10-06 13:03:10 EDT --- Could we get some more information on this? How many libraries are included? How many are dead upstream and how many won't accept brl-cad's patches? Where were the patches sent to? Where the libraries are dead upstream, the best thing to do would be for brl-cad upstream to take over (or release a renamed fork) with their modified versions. Where the upstreams don't want the changes we should find out why (a) the upstreams don't want them and (b) why brl-cad does. Then we'd have to decide whether to work with brl-cad to phase out the need for the library, work with the library upstream to clean up the changes for inclusion, or get someone (hopefully brl-cad) to maintain the forked version. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 Christopher Sean Morrison brl...@mac.com changed: What|Removed |Added CC||brl...@mac.com --- Comment #11 from Christopher Sean Morrison brl...@mac.com 2009-10-06 16:33:16 EDT --- We wish we didn't have to do that kind of thing too. Alas, there is not a universal cross-platform package management system that we can rely on to be default installed cross-platform. With the exception of openNURBS and NIST SCL, our external dependencies are unmodified and primarily provided as a download convenience for users. Our build aims hard to just work by default, regardless of platform and environment. We go to great lengths in our build system to perform compilation testing to detect installed libraries and to use them when we can. Additionally, there are compilation options to force all or individual dependencies on and off so that package management systems can be sure they're getting a system-installed library. The default is merely auto-detect. I'm really not sure what Matt meant by not following standard naming schemes. Is there even such a thing? The problem has been simple naming conflicts as our core public libraries are sub-projects in themselves with trade mark identity. The solution there is to install our libraries in a sub-directory (e.g., /usr/lib/brlcad/librt.so) and update the system linker search paths so the conflict is avoided. To get more specific on our external dependencies and respond to Toshio's request .. here's the list of installable dependencies their status: tkhtml3: no source changes, build convenience tkImg: no source changes, build convenience Utah Raster Toolkit: no upstream, we apply security and build fixes (but otherwise do not modify) TNT: no source changes (it's only header files) openNURBS: some source mods, upstream is not interested (competes with their business) NIST SCL: no upstream, heavily modified (we'll eventually manage it as a sub-project) We're using openNURBS in a way that upstream specifically doesn't support. It's a fantastic library that provides (a) geometric representation and (b) conversion facilities, yet is a subset of their larger commercial Rhino SDK which includes (c) geometric analysis facilities. We need a, b, and c for the same reasons they did, so we have to implement a portion of what they intentionally remove. We're looking into ways to refactor our modifications so they are an independent superset (so upstream is unmodified), but that's not where things are at today. Big thanks and appreciation to Matt Chan for taking this up. Thanks to everyone else for taking the time to review and critique. It will be great to see BRL-CAD integrated. Cheers! Sean -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949
--- Comment #12 from Nicolas Chauvet (kwizart) kwiz...@gmail.com 2009-10-06
16:44:05 EDT ---
There is also a problem with the librt.so from brlcad that may conflicts with
the glibc one on linux. Any progress on this side ? (specially using pkg-config
will be an easy way to abstract the problem).
At least that was the pending question when I took care of the package some
time ago:
https://bugzilla.redhat.com/show_bug.cgi?id=236856
There are also well known duplicates: (took from my spec file).
#Rename wall
#mv $RPM_BUILD_ROOT%{_bindir}/wall $RPM_BUILD_ROOT%{_bindir}/brlcad-wall
mv $RPM_BUILD_ROOT%{_bindir}/dstat $RPM_BUILD_ROOT%{_bindir}/brlcad-dstat
mv $RPM_BUILD_ROOT%{_bindir}/istat $RPM_BUILD_ROOT%{_bindir}/brlcad-istat
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #13 from Christopher Sean Morrison brl...@mac.com 2009-10-06 17:16:26 EDT --- Ah yes, can't forget the *long* efforts of kwizart that started us down this path (thanks to you too Nicolas!).. The 'wall' command was renamed about 20 months ago. I don't recall dstat/istat ever being raised as an issue, though. Do you have a reference link for the istat conflict? Regardless, dstat and istat are non-critical tools that can be easily renamed. I'll put it in our queue for the next release (7.16.2). We provide pkg_config files (as well as a brlcad-config script) so installing libraries into a subdirectory should take care of the librt/libbu/libbn conflicts. Cheers! Sean -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added CC||ke...@tummy.com --- Comment #5 from Kevin Fenzi ke...@tummy.com 2009-09-06 00:28:39 EDT --- Hey Matt. I took a quick look, and a few things to address before a review: 1. It looks like your are building some of the internally bundled copies of libraries: Build tkhtml3 : yes Build tkImg ..: yes Build Utah Raster Toolkit.: yes Build Template Numerical Toolkit..: yes Build openNURBS...: yes Build NIST STEP Class Libraries...: yes You should use system versions of these, or in cases they don't yet exist in Fedora, submit them for review first. I know tkImg at least is in Fedora already, not sure about the others. 2. The License tag doesn't appear right... see the Licesing page for the correct tags, and note that , is not valid. 3. rpmlint has a number of complaints. Try and address those? If you can take a look at those and spin up a new package I can look at reviewing this for you. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 --- Comment #3 from matt chan talc...@gmail.com 2009-08-25 03:36:02 EDT --- Thanks for the quick comments! I've made the changes to add sparc64, ppc64, and alpha to the %ifarch operator The spec file at the link has been updated. The new SRPM is at http://mattchan.homelinux.net:5/brlcad/brlcad-7.14.9.20090823svn-1.fc11.src.rpm Matt -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 Martin Gieseking martin.giesek...@uos.de changed: What|Removed |Added CC||martin.giesek...@uos.de Blocks||177841(FE-NEEDSPONSOR) -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added CC||kwiz...@gmail.com --- Comment #1 from Jason Tibbitts ti...@math.uh.edu 2009-08-24 10:39:37 EDT --- *** Bug 236856 has been marked as a duplicate of this bug. *** -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 518949] Review Request: brlcad - computer aided solid modelling and design
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=518949 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added CC||lemen...@gmail.com --- Comment #2 from Peter Lemenkov lemen...@gmail.com 2009-08-24 10:47:45 EDT --- Matt, you're using x86_64 as a marker for 64-bt arch - please, keep in mind, that we also have sparc and ppc64 targets. I'll post more notes later. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
