Opened 10 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)
Change History (8)
comment:1 Changed 10 years ago by mf2k (Frank Schima)
| Keywords: | mediainfo curl with-libcurl removed |
|---|---|
| Version: | 2.2.1 |
comment:2 follow-up: 3 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
comment:3 Changed 10 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 10 years ago by Pesthauch666@…
| Attachment: | patch-CLI_Compile.sh.diff added |
|---|
comment:4 Changed 10 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 10 years ago by ctreleaven (Craig Treleaven)
| Attachment: | patch-mediainfo_2014Apr02_01.diff added |
|---|
Replaces previous
comment:6 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
update to 0.7.70 + your patches for curl r125312
Sure, let's enable curl all the time. Can you provide a patch?
GUIs would be better implemented as subports not variants.