Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 181783 - app-arch/zip-2.32: "crypt" USE flag has no effect (+Ignored {C,LD}FLAGS)
Summary: app-arch/zip-2.32: "crypt" USE flag has no effect (+Ignored {C,LD}FLAGS)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-12 16:30 UTC by Arfrever Frehtes Taifersar Arahesis (RETIRED)
Modified: 2007-06-13 00:30 UTC (History)
0 users

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


Attachments
Patch (zip.patch,730 bytes, patch)
2007-06-12 19:49 UTC, Arfrever Frehtes Taifersar Arahesis (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-06-12 16:30:03 UTC
CFLAGS are always ignored.

### List of files built with LDFLAGS ignored ###
/usr/bin/zip
/usr/bin/zipcloak
/usr/bin/zipsplit
/usr/bin/zipnote
### End of list                              ###
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2007-06-12 17:48:20 UTC
OK, that was about enough.

1/ Use common sense when filing bugs (like, what about mailing gentoo-dev mailing list first and asking about the best course of action here?
2/ If you have a patch, submit it *upstream* unless this is a *Gentoo-specific* issue.
Comment 2 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-06-12 18:14:58 UTC
This bug report also concerns CFLAGS. Unfortunately you didn't see it.
http://devmanual.gentoo.org/ebuild-writing/functions/src_compile/building/index.html and http://devmanual.gentoo.org/general-concepts/user-environment/index.html say that CFLAGS should be respected. I think that this bug report should be reopened.
Comment 3 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-06-12 19:48:32 UTC
I discovered that target "generic" has broken behaviour (and generally this Makefile is strange). It uses variables from file "flags", so this file must be recreated.

If "eval $(MAKE) zips `cat flags`" is changed to "eval $(MAKE) zips", then we have new problems:
 1. CFLAGS doesn't contain probably necessary -D* flags.
 2. "LFLAGS1 =" line should be changed to "LFLAGS1 = $(LDFLAGS)" using sed.
 3. "LFLAGS2 = -s" line causes "QA Notice: Pre-stripped files found".
 4. OBJA isn't set and executable "zip" lacks code from 2 object files, because there is in unix/Makefile:
OBJA =
...
zip$(EXE): $(OBJZ) $(OBJI) $(OBJA)
        $(BIND) -o zip$(EXE) $(LFLAGS1) $(OBJZ) $(OBJI) $(OBJA) $(LFLAGS2)



I think that recreating file "flags" is good decision.
Comment 4 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-06-12 19:49:53 UTC
Created attachment 121847 [details, diff]
Patch
Comment 5 SpanKY gentoo-dev 2007-06-13 00:30:15 UTC
that patch is certainly not right ... you cant hardcode those flags in the ebuild

ive cleaned up the build process in cvs