Ticket #328 (closed enhancement: fixed)

Opened 10 years ago

Last modified 9 years ago

Update mc.ext in respect to the handling of XIPH multimedia formats .ogg, oga, .ogv, etc

Reported by: jemm Owned by: styx
Priority: major Milestone: 4.7
Component: mc-core Version: 4.6.2
Keywords: commited-master Cc:
Blocked By: Blocking:
Branch state: Votes for changeset:

Description

As defined in http://wiki.xiph.org/index.php/MIME_Types_and_File_Extensions I'd propose to update mc.ext as follows:

update line 381 from:

regex/\.([oO][gG][gG])$

to:

regex/\.([oO][gG][gG|aA|vV|xX])$

And add thereafter

regex/\.([sS][pP][xX]|[fF][lL][aA][cC])$

Open=if [ "$DISPLAY" = "" ]; then play %f; else (xmms %f >/dev/null 2>&1 &); fi

The whole section is called ### Sound files ### and if having .ogv and .ogx (video files) there is unsuitable, then maybe ### Video files ### could be added. However the point is that ogginfo is perfectly capable of showing content of all multimediafiles which use .ogg container and that will help a lot of users who deal with these files in commandline.

Change History

comment:1 Changed 10 years ago by jemm

  • Summary changed from Update to XIPH multimedia formats .ogg, oga, .ogv, etc to Update mc.ext in respect to the handling of XIPH multimedia formats .ogg, oga, .ogv, etc

comment:2 Changed 10 years ago by angel_il

  • Type changed from defect to enhancement

comment:3 Changed 10 years ago by styx

  • Owner set to styx
  • Status changed from new to accepted

comment:4 Changed 10 years ago by styx

  • Keywords review added

branch: 328_mc_ext_xiph
changeset: dafaaa4653dece699eba99e39ca2d41f0ee8d66f

comment:5 Changed 10 years ago by angel_il

  • Keywords vote-angel_il added

comment:6 Changed 10 years ago by slavazanko

  • Keywords vote-slavazanko approved added; review removed

comment:7 Changed 10 years ago by styx

  • Status changed from accepted to testing
  • Keywords commited-master added; vote-angel_il vote-slavazanko approved removed
  • Resolution set to fixed

comment:8 Changed 10 years ago by styx

  • Status changed from testing to closed

comment:9 Changed 10 years ago by angel_il

Note: See TracTickets for help on using tickets.