[avrdude-dev] [patch #7792] Extends libelf header checks to libelf/libelf.h

[Hannes Weisbach]

URL:
  <http://savannah.nongnu.org/patch/?7792>

                 Summary: Extends libelf header checks to libelf/libelf.h
                 Project: AVR Downloader/UploaDEr
            Submitted by: hweisbach
            Submitted on: Mi 13 Jun 2012 20:40:33 GMT
                Category: None
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any

    _______________________________________________________

Details:

Using MacPorts under OS X 10.6.8 (and I believe this applies to any version of
OS X), the header files for the libelf package are installed in
$prefix/libelf/libelf.h. So the configure script checks first, if it can link
against libelf, which succeeds, since the library is located in $prefix/lib.
After that, configure checks for libelf.h - which is not found.
My patch adds looking for libelf.h and libelf/libelf.h. fileio.c, the only
file which includes libelf.h was edited accordingly to include either libelf.h
or libelf/libelf.h - which ever was found.
I should say, I have no experience with autoconf and I may have taken the
wrong approach here. Maybe an autoconf-wiz knows how to handle this situation
appropriately.

The only remaining problem is, that code depending on HAVE_LIBELF requires
actually HAVE_LIBELF_H, because it uses declarations (types, functions) from
that file. I admit, having the library but not the header file might be a rare
corner case, but I thought I should mention it anyway.

Best regards,
Hannes



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Mi 13 Jun 2012 20:40:33 GMT  Name: libelf_h.patch  Size: 722B   By:
hweisbach

<http://savannah.nongnu.org/patch/download.php?file_id=26021>

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?7792>

_______________________________________________
  Nachricht gesendet von/durch Savannah
  http://savannah.nongnu.org/