Opened 11 years ago

Closed 11 years ago

#19962 closed defect (fixed)

vlc +aqua won't compile

Reported by: nefar@… Owned by: max@…
Priority: Normal Milestone:
Component: ports Version: 1.7.1
Keywords: Cc: nerdling (Jeremy Lavergne)
Port: vlc

Description

I got libxml errors during compile. I compiled several other ports in between, one of which compiled libxml2. When I then came back to vlc, it now compiled fine. Sorry to be so vague, but that's what I've got.

Attachments (2)

vlc.install.log (887.8 KB) - added by nefar@… 11 years ago.
The log file for failed build
VLC-0.9.9a-aqua+libxml2.diff (1.8 KB) - added by max@… 11 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 11 years ago by tobypeterson

Resolution: invalid
Status: newclosed

uh, yeah

Changed 11 years ago by nefar@…

Attachment: vlc.install.log added

The log file for failed build

comment:2 Changed 11 years ago by jmroot (Joshua Root)

Port: vlc added
Resolution: invalid
Status: closedreopened

comment:3 Changed 11 years ago by jmroot (Joshua Root)

Owner: changed from macports-tickets@… to max@…
Status: reopenednew

-I/usr/include/libxml2 looks rather suspect.

comment:4 in reply to:  1 ; Changed 11 years ago by nefar@…

Replying to toby@…:

uh, yeah

btw, this was rather easily reproduced. I simply moved the /opt/local to /opt/local.old etc (same for all macports install dirs) and tried it over again. I would actually have expected more of a serious response than "uh, yeah" and closing the ticket as invalid without doing anything at all to investigate.

comment:5 Changed 11 years ago by jmroot (Joshua Root)

And indeed, neither vlc nor its dependencies declare a dependency on libxml2.

comment:6 Changed 11 years ago by mf2k (Frank Schima)

Resolution: fixed
Status: newclosed

I confirmed that libxml2 was required and I added the dependency in r52437.

Changed 11 years ago by max@…

comment:7 Changed 11 years ago by max@…

Yes, the libxml2 dependency is missing unfortunatly. Thanks for reporting this.

The patch also removes the aqua variant and moves it's functionality into platform macosx, as discussed in ticket 19325

comment:8 Changed 11 years ago by mf2k (Frank Schima)

Resolution: fixed
Status: closedreopened

comment:9 in reply to:  4 Changed 11 years ago by tobypeterson

Replying to nefar@…:

Replying to toby@…:

uh, yeah

btw, this was rather easily reproduced. I simply moved the /opt/local to /opt/local.old etc (same for all macports install dirs) and tried it over again. I would actually have expected more of a serious response than "uh, yeah" and closing the ticket as invalid without doing anything at all to investigate.

In its original form, this bug report was completely useless. While it may have been easily reproducible, bug reports that take the form "it doesn't work" are not acceptable.

comment:10 Changed 11 years ago by ryandesign (Ryan Schmidt)

Cc: snc@… added
Resolution: fixed
Status: reopenedclosed

The patch was partially applied in r53864 with the update to 1.0.0_0; the part that wasn't committed prevented VLC.app from being installed. I committed the last bit in r53998 and bumped the revision to 1.0.0_1 so everyone will again have VLC.app.

Note: See TracTickets for help on using tickets.