Ticket #3122 (closed defect: fixed)

Opened 10 years ago

Last modified 10 years ago

xorriso iso9660 show only one depth

Reported by: holbru Owned by: slavazanko
Priority: major Milestone: 4.8.12
Component: mc-vfs Version: master
Keywords: Cc: dnh@…
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description

GNU Midnight Commander 4.8.11

xorriso show only one depth, I can change the dir into the iso but no content available

extfs.d/iso9660

the patch solve that problem

Attachments

iso9660.patch (566 bytes) - added by holbru 10 years ago.
iso9660 patch for xorriso
mc-extfs-iso9660-xorriso.patch (1.2 KB) - added by dnh 10 years ago.
holbru's patch is a bit incomplete. At least I found some more problems, esp. with DVD (UDF) images ... This patch falls back on isoinfo.
mc-extfs-iso9660-xorriso.2.patch (1.3 KB) - added by wd 10 years ago.
some cosmetics
mc-extfs-iso9660-xorriso.3.patch (1.1 KB) - added by dnh 10 years ago.
mc-extfs-iso9660-xorriso.3.patch
iso9660.diff (651 bytes) - added by holbru 10 years ago.

Change History

Changed 10 years ago by holbru

iso9660 patch for xorriso

Changed 10 years ago by dnh

holbru's patch is a bit incomplete. At least I found some more problems, esp. with DVD (UDF) images ... This patch falls back on isoinfo.

comment:1 Changed 10 years ago by dnh

  • Cc dnh@… added

Changed 10 years ago by wd

some cosmetics

Changed 10 years ago by dnh

mc-extfs-iso9660-xorriso.3.patch

comment:2 Changed 10 years ago by dnh

Those last cosmetics from wd (second block) are not portable to e.g. ash, dash, busybox sh ... the ".3.patch" is against "HEAD" from "now" (last change: commit 5f94d51fab5572ded65208629ef4730fead4a712 from Fri Feb 28 13:33:36 2014 +0400).

comment:3 Changed 10 years ago by slavazanko

  • Status changed from new to accepted
  • Owner set to slavazanko
  • Milestone changed from Future Releases to 4.8.12

comment:4 Changed 10 years ago by slavazanko

  • Branch state changed from no branch to on review

Created branch 3122_xorriso_one_depth. Initial changeset:fb45995c734066af251dd15fc23fbd15bef7ee8c

Review please.

Changed 10 years ago by holbru

comment:5 follow-up: ↓ 6 Changed 10 years ago by holbru

The 'lsl' must contain only the clear value, nothing else.

echo $lsl | /bin/grep , crasht if the line is empty

iso9660.diff

comment:6 in reply to: ↑ 5 Changed 10 years ago by holbru

Replying to holbru:

The 'lsl' must contain only the clear value, nothing else.

echo $lsl | /bin/grep , crasht if the line is empty

iso9660.diff

update: it works without the last patch. Sorry, Thanks for the work.

comment:7 Changed 10 years ago by andrew_b

  • Votes for changeset set to andrew_b
  • Component changed from mc-core to mc-vfs
  • Branch state changed from on review to approved

comment:8 Changed 10 years ago by slavazanko

  • Votes for changeset changed from andrew_b to committed-master
  • Branch state changed from approved to merged

comment:9 Changed 10 years ago by slavazanko

  • Status changed from accepted to testing
  • Resolution set to fixed

comment:10 Changed 10 years ago by slavazanko

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.