Opened 3 years ago

Closed 3 years ago

#41603 closed defect (fixed)

minidlna: upgrade to latest version 1.1.1 fails

Reported by: mkae (Marko Käning) Owned by: ranauei@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: minidlna

Description

I tried to upgrade to the latest version of minidlna using my attached portfile patch, but failed pretty early during build:

  CC       getifaddr.o
getifaddr.c: In function ‘getifaddr’:
getifaddr.c:85: error: ‘IFF_SLAVE’ undeclared (first use in this function)
getifaddr.c:85: error: (Each undeclared identifier is reported only once
getifaddr.c:85: error: for each function it appears in.)

Any idea what could be done about it?

Attachments (3)

patch-getifaddr.c.diff (278 bytes) - added by mkae (Marko Käning) 3 years ago.
patch to fix the IFF_SLAVE
Portfile (1.3 KB) - added by mkae (Marko Käning) 3 years ago.
patch for minidlna's portfile including the IFF_SLAVE patch
minidlna_Portfile.diff (1.2 KB) - added by mkae (Marko Käning) 3 years ago.
THIS is the patch

Download all attachments as: .zip

Change History (9)

Changed 3 years ago by mkae (Marko Käning)

patch to fix the IFF_SLAVE

Changed 3 years ago by mkae (Marko Käning)

patch for minidlna's portfile including the IFF_SLAVE patch

Changed 3 years ago by mkae (Marko Käning)

THIS is the patch

comment:1 Changed 3 years ago by mkae (Marko Käning)

ranauei, this works for me on Snow Leopard.

Can you test it on another version of Mac OSX?

Could be newer versions wouldn't need the included patch for getifaddr.c?

comment:2 Changed 3 years ago by mkae (Marko Käning)

I can test build on 10.9 myself tomorrow...

Watching streams with minidlna running on 10.6.8 works fine with a Samsung TV.

comment:3 Changed 3 years ago by mkae (Marko Käning)

The patch is also needed on 10.9.

So this should be reported upstream, I guess...

comment:4 Changed 3 years ago by mkae (Marko Käning)

comment:5 Changed 3 years ago by mkae (Marko Käning)

ranauei, if you do not object I will commit the patched upgrade to MacPorts.

comment:6 Changed 3 years ago by mkae (Marko Käning)

  • Resolution set to fixed
  • Status changed from new to closed

Done in r114826

Note: See TracTickets for help on using tickets.