Opened 11 years ago

Closed 10 years ago

#42424 closed defect (fixed)

avrdude: Opportunistic use of libelf headers

Reported by: raimue (Rainer Müller) Owned by: bd34n (Brian Dean)
Priority: Normal Milestone:
Component: ports Version: 2.2.1
Keywords: Cc: cooljeanius (Eric Gallager), macports@…
Port: avrdude

Description

avrdude uses libelf headers if the port is installed prior to compilation, but does not declare the appropriate dependency.

With libelf:

$ port -q installed \( libelf avrdude \) and active
  avrdude @6.0.1_0 (active)
  libelf @0.8.13_2 (active)
$ sudo port -v configure avrdude
[...]
Configuration summary:
----------------------
DO HAVE    libelf
[...]

Without libelf:

$ port -q installed \( libelf avrdude \) and active
  avrdude @6.0.1_0 (active)
$ sudo port -v configure avrdude
[...]
Configuration summary:
----------------------
DON'T HAVE libelf
[...]

This means the resulting binary changes whether libelf was active at the time of compilation or not. Builds are meant to be reproducible, they should not change due to factors not declared in the Portfile.

Attachments (1)

avrdude-libelf.patch (924 bytes) - added by macports@… 11 years ago.
Patch to enforce the libelf dependency

Download all attachments as: .zip

Change History (6)

comment:1 Changed 11 years ago by raimue (Rainer Müller)

Summary: avrdude: Opportunistic linking against libelfavrdude: Opportunistic use of libelf headers

comment:2 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

Changed 11 years ago by macports@…

Attachment: avrdude-libelf.patch added

Patch to enforce the libelf dependency

comment:3 Changed 11 years ago by macports@…

The easiest solution would be to enforce a dependency on libelf (see attached patch). Chances are probably pretty low that someone wants to use avrdude without ELF support, so we don't need an extra variant.

comment:4 Changed 11 years ago by macports@…

Cc: macports@… added

Cc Me!

comment:5 Changed 10 years ago by g5pw (Aljaž Srebrnič)

Resolution: fixed
Status: newclosed

Fixed by explicitly depending on libelf in r120473.

Note: See TracTickets for help on using tickets.