Opened 14 years ago

Closed 14 years ago

#27968 closed defect (fixed)

phonon @4.4.2 fails to build on Tiger

Reported by: roberto@… Owned by: michaelld (Michael Dickens)
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: tiger Cc: ryandesign (Ryan Carsten Schmidt)
Port: phonon

Description

This port does not build on my system. Please see attached build log.

Attachments (1)

main.log.bz2 (1.7 KB) - added by roberto@… 14 years ago.

Download all attachments as: .zip

Change History (7)

Changed 14 years ago by roberto@…

Attachment: main.log.bz2 added

comment:1 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… michaelld@… added
Keywords: tiger added
Summary: phonon @4.4.2 fails to buildphonon @4.4.2 fails to build on Tiger

The relevant error message in the log says:

Target org.macports.patch returned: find: 1: unknown expression primary

It is also relevant that you are on Tiger, because in the post-patch phase, phonon is trying to run a command like:

find ${worksrcpath}/phonon -name "*.h" -depth 1

And "-depth 1" is not a valid argument to the version of "find" that is on Tiger.

See #26034 for how this same problem was fixed in the qt4-mac port.

comment:2 Changed 14 years ago by michaelld (Michael Dickens)

Owner: changed from macports-tickets@… to michaelld@…

Well, that's certainly my bad. Working on it ...

comment:3 Changed 14 years ago by michaelld (Michael Dickens)

Cc: michaelld@… removed

Cc Me!

comment:4 Changed 14 years ago by roberto@…

So, I looked at #26034 like you recommended. However, it left me scratching my head and trying to figure out why you didn't just replace 'find <blah> -depth 1' with 'find <blah> -maxdepth 1 -mindepth 1'. I made that change in the phonon Portfile on my system and it worked just fine. (My apologies for not looking more closely at the initial build log, else I would have recommended the fix in my initial ticket submission.)

comment:5 Changed 14 years ago by michaelld (Michael Dickens)

I had no idea 'find' in 10.4 supported "maxdepth" and "mindepth"; I'll make that change pronto after a quick test to make sure it's compatible with 10.6 ... should be. Thanks for the quick fix!

comment:6 Changed 14 years ago by michaelld (Michael Dickens)

Resolution: fixed
Status: newclosed

Fixed in r75279.

Note: See TracTickets for help on using tickets.