[barcode] DESTDIR support patch for GNU barcode

Daniel Macks dmacks@netspace.org
Mon Feb 27 15:48:48 CET 2006


On Sun, Feb 26, 2006 at 01:11:07AM -0800, Claudio Fontana wrote:
> 
> I am providing here a patch to support DESTDIR in GNU
> barcode build system.
[...]
> I would also suggest avoiding the use of directories
> as the second argument of install scripts invocation,
> because it can have undesired effects in some edge
> conditions (ref: GNU coding standards 7.2.3 Variables
> for Specifying Commands).
> 
> Attached: barcode.DESTDIR_support.p0.udiff
> Apply from $top_srcdir with:
> patch -p0 < barcode.DESTDIR_support.p0.udiff

Switching to 7.2.3 compliance reveals a weakness in how barcode.info
is handled:

INFO = doc/$(TARGET).info
$(INSTALL) -c -m 0644  $(INFO) $(DESTDIR)$(INFODIR)/$(INFO)

so it goes into $INFODIR/doc/ not $INFODIR itself. Better to have the
$(INSTALL) command specify the local (source) location, not the
variables:

INFO = $(TARGET).info
$(INSTALL) -c -m 0644  doc/$(INFO) $(DESTDIR)$(INFODIR)/$(INFO)

dan

-- 
Daniel Macks
dmacks@netspace.org
http://www.netspace.org/~dmacks



More information about the barcode mailing list