[Bug 199168] Review Request: CGAL

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: CGAL


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199168





------- Additional Comments From laurent.rineau__fedora_extras@xxxxxxxxxxxxxx  2006-08-16 03:58 EST -------
(In reply to comment #10)
> > CGAL is a meta-package that requires CGAL-libs, CGAL-devel, and 
CGAL-sources. 
> 
> Contradicts Fedora conventions and IMNSOH, is complete non-sense.
> Consider this to be a MUST FIX.
> 
> Put the run-time libs into CGAL or CGAL-libs and the devel files into 
*-devel.

For the moment (CGAL-3.2.1-14), libs are in CGAL-libs, and devel files are in 
CGAL-devel. 

> > > E: CGAL-devel script-without-shellbang /etc/profile.d/cgal.sh
> > > E: CGAL-devel script-without-shellbang /etc/profile.d/cgal.csh
> > 
> > As far as I know, these rpmlint errors should be ignored.
> Nope, these scripts are incomplete. MUSTFIX

Ok. I thought it was agreed that shell scripts in /etc/profile.d/ should not 
have shell bangs. If it is really a MUSTFIX, this should be written somewhere, 
and bugs should be reported, against all almost all package that ship 
something in /etc/profile.d/

> > (In reply to comment #8)
> > > Further issues:
> > > 
> > > - The *-devel package ships /usr/include/CORE
> > > IMO, this directory name is too general.
> > 
> > CGAL-3.2.1 ships CORE-1.7, 
http://www.cs.nyu.edu/exact/core_pages/intro.html
> > This directory is from CORE.
> And? This doesn't answer my remark.

I do not see any solution, here. <CORE/...h> is the documented way to include 
CORE headers. If this is a blocker, CORE cannot be into Fedora. That's it.
> > > - A more general design problem:
> > > Some headers in /usr/include/CGAL hard-code configuration-time detected
> > > * system features, e.g. the version of zlib and Qt
> > > 
> > > * compiler characteristics, e.g. endianness.
> > 
> > Yes, it should only be /usr/include/CGAL/compiler_config.h. Is it a 
blocker?
> Well, there actually are 2 issues with this.
> - Package dependencies. You will have to find a way to handle the hard-coded
> version dependencies in rpm.

Actually, these version macros are not used in CGAL. They are not even 
documented. They could be pruned. They are used by the CGAL test suite to 
display dependencies' versions.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]