Opened 9 years ago

Closed 9 years ago

#47595 closed enhancement (fixed)

mlt: add universal variant

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: ddennedy (Dan Dennedy)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: mlt

Description

A universal variant should be added to the mlt port. The reason given in the port for why it is disabled is:

# frei0r-plugins is not universal
universal_variant   no

This is no longer correct; frei0r-plugins does have a universal variant now.

Just removing these lines is not enough because mlt is not currently using -arch flags, which it should be doing anyway, even for non-universal builds.

Just adding -arch flags to CFLAGS and LDFLAGS isn't enough either, because then the universal build fails, saying:

cannot use 'dependencies' output with multiple -arch options

To work around that we may need to use the muniversal portgroup.

Change History (4)

comment:1 Changed 9 years ago by ddennedy (Dan Dennedy)

Sorry, but I am not volunteering to fix this. There was a patch submitted to the mailing list, but it interfered with builds for other platforms. It looks there is still a lot of changes needed in the build system to accommodate it.

Last edited 9 years ago by ddennedy (Dan Dennedy) (previous) (diff)

comment:2 Changed 9 years ago by ddennedy (Dan Dennedy)

Here is a link to the mailing list thread and patch if someone wants to finish the work: https://www.mail-archive.com/mlt-devel@lists.sourceforge.net/msg04058.html

Alternatively, it could be a patch for the MacPorts mlt Port instead of merged upstream.

Last edited 9 years ago by ddennedy (Dan Dennedy) (previous) (diff)

comment:3 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)

Ah! Thanks for the pointer to the existing mailing list discussion. I agree with the analysis that the cause of the error is using CFLAGS where CPPFLAGS should have been used, but I see fixing it is more involved.

I'm trying to instead work around the problem by using the muniversal portgroup. I'm running into some problems but they're related to the portgroup, not mlt. Hopefully I can figure out that problem soon and get it working; I'll attach a diff here for your approval when I do.

I'd really like to get a universal variant for mlt, because the synfig port, which I maintain, will depend on mlt in the next version, and I want synfig to continue to have a universal variant then.

comment:4 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.