Opened 11 years ago

Closed 10 years ago

#42269 closed enhancement (fixed)

mediainfo @0.7.64 Add libcurl support

Reported by: Pesthauch666@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: kurthindenburg (Kurt Hindenburg)
Port: mediainfo

Description

After installing mediainfo via MacPorts some while ago I noticed that by default only local files can be viewed, which makes it rather useless for me, because I'm using it mainly to check recorded tv-streams on a Dreambox via it's web-interface.

Since than, I switched to a manual compilation/installation (which I usually try to avoid) using the --with-libcurl configuration option of 'MediaInfoLib', by editing the 'CLI_Compile.sh'. Since this is one of the few tools that I couldn't use with a MacPorts installation, wouldn't it be better to include the libcurl support either by default or as a new variant?

And another idea for a new variant:
Besides that I'm only using the CLI version of the program anyway, maybe for some others a variant for the GUI built could become useful.

... and btw the most recent version is 0.7.67

Thanks in advance

Attachments (2)

patch-CLI_Compile.sh.diff (798 bytes) - added by Pesthauch666@… 11 years ago.
patch-mediainfo_2014Apr02_01.diff (1.9 KB) - added by ctreleaven (Craig Treleaven) 11 years ago.
Replaces previous

Download all attachments as: .zip

Change History (8)

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

Keywords: mediainfo curl with-libcurl removed
Version: 2.2.1

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

Sure, let's enable curl all the time. Can you provide a patch?

GUIs would be better implemented as subports not variants.

comment:3 in reply to:  2 Changed 11 years ago by Pesthauch666@…

Replying to ryandesign@…:

Sure, let's enable curl all the time. Can you provide a patch?

Well, I'm new to this, so I can't guarantee for the result, but I'm giving it a try. I hope I did it right by applying the the already provided patch first, before I did my changes?

Changed 11 years ago by Pesthauch666@…

Attachment: patch-CLI_Compile.sh.diff added

comment:4 Changed 11 years ago by ctreleaven (Craig Treleaven)

Bump to 0.7.67, correct master_sites, build with libcurl support. Tested OK on 10.6.8.

Changed 11 years ago by ctreleaven (Craig Treleaven)

Replaces previous

comment:5 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:6 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed

update to 0.7.70 + your patches for curl r125312

Note: See TracTickets for help on using tickets.