search

krio-rogue / ioping

Automatically exported from code.google.com/p/ioping
GNU General Public License v3.0
0 stars 0 forks source link

Enhancement of Makefile #7

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Several enhancements for Makefile:

* Drop -g from default CFLAGS, as debugging build shouldn't be the default build
* exchange LDFLAGS with LIBS, LDFLAGS should only contain linker directives, 
where as LIBS contain the libs to be linked with
* Let PREFIX to be easily overwriten

All those changes are necessary for packing ioping easily for distributions.

Please include

Original issue reported on code.google.com by ju.leche...@googlemail.com on 13 Jan 2012 at 8:10

Attachments:

GoogleCodeExporter commented 9 years ago 
* debug can be stripped by strip or install
* for what?
* does not required, make ignore assignments if variable overridden in 
commandline

Original comment by koct9i on 12 Feb 2012 at 12:29

GoogleCodeExporter commented 9 years ago 
* right, but therefore you need to strip it. For general usage -g isn't 
required.
* Most importantly for linking with -Wl,--as-needed enabled. In order to allow 
the linker to optimize the order must be LINKER LDFLAGS OBJ LIBS. For more 
reading I will refer to http://blog.flameeyes.eu/tag/linker

Original comment by ju.lec...@googlemail.com on 12 Feb 2012 at 7:22

GoogleCodeExporter commented 9 years ago

Original comment by koct9i on 25 Aug 2012 at 3:36

GoogleCodeExporter commented 9 years ago

Original comment by koct9i on 28 Feb 2013 at 7:53