Bug 1102855 - glpk bundles zlib
glpk bundles zlib
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: glpk (Show other bugs)
rawhide
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Conrad Meyer
Fedora Extras Quality Assurance
:
Depends On:
Blocks: DuplicSysLibsTracker
  Show dependency treegraph
 
Reported: 2014-05-29 13:11 EDT by Ville Skyttä
Modified: 2014-05-29 15:29 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2014-05-29 15:29:08 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Ville Skyttä 2014-05-29 13:11:31 EDT
https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
https://fedoraproject.org/wiki/Packaging:Treatment_Of_Bundled_Libraries

The bundled zlib in this case appears to be a somewhat modified one.
Comment 1 Conrad Meyer 2014-05-29 13:51:27 EDT
> The bundled zlib in this case appears to be a somewhat modified one.

From README in src/zlib, it seems more like a fork than copy-paste bundling:

> NOTE: Files in this subdirectory are NOT part of the GLPK package, but
>       are used with GLPK.
> 
>       The original code was modified according to GLPK requirements by
>       Andrew Makhorin <mao@gnu.org>.
> 
>       The following files were rewritten:
>       gzguts.h, zconf.h, zutil.h.
> 
>       The following files were added:
>       zio.h, zio.c.
> 
>       Other files were not changed.

But I have not inspected the source files.
Comment 2 Conrad Meyer 2014-05-29 13:56:55 EDT
It seems like their fork does a couple things:

- Tries to hide zlib symbols in _glp_zlib namespace to prevent linking collisions (zconf.h)
- Wraps POSIX fd IO in C stdlib FILE IO ... for compatibility? Fedora doesn't need it, certainly (zio.h, zio.c)

Not sure what they modified in zutil.h or gzguts.h.
Comment 3 Conrad Meyer 2014-05-29 13:57:54 EDT
(The former could have been done with -fvisibility=hidden, *sigh*.)
Comment 4 Conrad Meyer 2014-05-29 14:01:19 EDT
My hunch is we can just kill the bundled zlib and link the system one.
Comment 5 Conrad Meyer 2014-05-29 14:08:46 EDT
From NEWS:

 173 GLPK 4.46 (release date: Aug 09, 2011)
...
 202         The zlib compression library (version 1.2.5) was ANSIfied,
 203         modified according to GLPK requirements and included in the
 204         distribution as an external software module. Thus, now this
 205         feature is platform independent.

So yeah, nothing we care about.
Comment 7 Conrad Meyer 2014-05-29 14:49:11 EDT
Build after adding BR on automake:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6909157
Comment 8 Conrad Meyer 2014-05-29 15:10:50 EDT
Added BR on libtool (autotools, why?!):
http://koji.fedoraproject.org/koji/taskinfo?taskID=6909199
Comment 9 Conrad Meyer 2014-05-29 15:16:22 EDT
One (cross your fingers) last one: zlib-devel.
http://koji.fedoraproject.org/koji/taskinfo?taskID=6909259

Note You need to log in before you can comment on or make changes to this bug.