Bug 157047

Summary: [Patch] graphics/imlib2 - Wrong LIB_DEPENDS if graphics/giflib is installed
Product: Ports & Packages Reporter: Herbert J. Skuhra <herbert>
Component: Individual Port(s)Assignee: Stanislav Sedov <stas>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Herbert J. Skuhra 2011-05-14 21:10:09 UTC
If graphics/giflib is installed graphics/imlib2 is linked against libgif.so.5
and not libungif.so.5!

% ls -l /var/db/pkg |grep gif
drwxr-xr-x  2 root  wheel  512 14 mai 21:25 giflib-4.1.6
drwxr-xr-x  2 root  wheel  512 13 mai 16:02 libungif-4.1.4_5

% pwd
/usr/ports/graphics/imlib2

% ldd work/imlib2-1.4.4/src/modules/loaders/.libs/gif.so |grep gif
work/imlib2-1.4.4/src/modules/loaders/.libs/gif.so:
        libgif.so.5 => /usr/local/lib/libgif.so.5 (0x281a4000)

So in this case LIB_DEPENDS is wrong.

Is the attached patch commitable or should we enforce linking against
libungif?

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2011-05-14 21:10:21 UTC
Responsible Changed
From-To: freebsd-ports-bugs->stas

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2011-05-15 00:09:55 UTC
stas        2011-05-14 23:09:46 UTC

  FreeBSD ports repository

  Modified files:
    graphics/imlib2      Makefile 
  Log:
  - Imlib picks up giflib if available instead of libungif.  Account for
    this.
  
  PR:             ports/157047
  Submitted by:   "Herbert J. Skuhra" <h.skuhra@gmail.com>
  
  Revision  Changes    Path
  1.141     +6 -1      ports/graphics/imlib2/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Stanislav Sedov freebsd_committer freebsd_triage 2011-05-15 00:10:08 UTC
State Changed
From-To: open->closed

Looks good!  Thanks!