Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 173907 - media-libs/libpng should install INSTALL file in documentation directory
Summary: media-libs/libpng should install INSTALL file in documentation directory
Status: VERIFIED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Gentoo Linux bug wranglers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-04-09 13:14 UTC by Gerben Vos
Modified: 2007-04-09 14:19 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gerben Vos 2007-04-09 13:14:32 UTC
During the build of some packages (like www-client/links), it is possible to get this build warning from pngconf.h:

/usr/include/libpng12/pngconf.h:317:2: warning: #warning __png.h__ already includes setjmp.h;
/usr/include/libpng12/pngconf.h:318:2: warning: #warning __dont__ include it again.;

pngconf.h then says:

     /* If you encounter a compiler error here, see the explanation
      * near the end of INSTALL.
      */

Therefore, it would be nice if INSTALL were added to the "dodoc" line in the libpng ebuild, so it is easier to look it up.

Also, maybe the www-client/links should be fixed so this warning disappears?

Reproducible: Always

Steps to Reproduce:
1. emerge links
2. Get warnings from pngconf.h
3. Read pngconf.h and find out that it refers you to the INSTALL file of libpng.

Actual Results:  
No /usr/share/doc/libpng-*/INSTALL to look up what's the problem.

Expected Results:  
Find an /usr/share/doc/libpng-*/INSTALL file.
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2007-04-09 13:18:06 UTC
We are not installing INSTALL files on purpose, because it's useless. It's already installed. 

If you want the warnings fixed, report them upstream: http://links.twibright.com/development.php

Thanks.
Comment 2 Gerben Vos 2007-04-09 14:09:42 UTC
(In reply to comment #1)
> We are not installing INSTALL files on purpose, because it's useless. It's
> already installed. 

But this INSTALL file includes information that is useful after the installation. Therefore, either it should be installed anyway, or the information should be moved to the README or some other file (and the pointer in the header file adjusted). I assume your opinion is that the latter should (also) be requested from upstream?
Comment 3 Jakub Moc (RETIRED) gentoo-dev 2007-04-09 14:19:43 UTC
As said above, we don't install this, by policy. Information useful after installing doesn't belong to INSTALL.