vlc: remove aqua variant

[ryandesign (Ryan Carsten Schmidt)]

Could the +aqua variant of the vlc port be removed? Just make what the variant does happen always on platform macosx. Having fewer variants is better.

[max@…]

What the +aqua variant does is create VLC.app and install it in ${applications_dir}. Without this variant there will be just a vlc executable in ${prefix}/bin. I think moving this to platform macosx wouldn't be appropriate. I could remove the variant at all if you feel it is unnecessary and always create/install VLC.app or rename it to something different

[ryandesign (Ryan Carsten Schmidt)]

Replying to max@…:

What the +aqua variant does is create VLC.app and install it in ${applications_dir}.

That's what I thought.

Without this variant there will be just a vlc executable in ${prefix}/bin.

Yes, I didn't mean you should remove this functionality.

I think moving this to platform macosx wouldn't be appropriate. I could remove the variant at all if you feel it is unnecessary and always create/install VLC.app or rename it to something different

My issue is not with the name of the variant, my issue is with there being a variant for this at all. Why would anyone want to turn it off? Just do it all the time, on Mac OS X.

My reason for suggesting doing this in the "platform macosx" section is so that it does not impact users installing vlc on other operating systems. However, I see the portfile already has other things in it that are Mac OS X-specific. I also see now that the "destroot.target" line already contains "install VLC.app" always. The aqua variant does nothing but append "VLC.app" to "destroot.target" again, so it's completely ineffective anyway. So you could either move all of the Mac OS X-specific stuff to the "platform macosx" section, or you could decide that the vlc port will only work on Mac OS X. In the latter case, you can simply remove the aqua variant, and also change the "platforms" line from "darwin" to "macosx".

[max@…]

Ok i moved the the functionality of +aqua into macosx and removed the standard destroot.target VLC.app

[(none)]

Milestone Port Enhancements deleted

[ryandesign (Ryan Carsten Schmidt)]

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.