Ticket #3122 (closed defect: fixed)

Opened 9 years ago

Last modified 9 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 9 years ago.
iso9660 patch for xorriso
mc-extfs-iso9660-xorriso.patch (1.2 KB) - added by dnh 9 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 9 years ago.
some cosmetics
mc-extfs-iso9660-xorriso.3.patch (1.1 KB) - added by dnh 9 years ago.
mc-extfs-iso9660-xorriso.3.patch
iso9660.diff (651 bytes) - added by holbru 9 years ago.

Change History

Changed 9 years ago by holbru

iso9660 patch for xorriso

Changed 9 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 9 years ago by dnh

  • Cc dnh@… added

Changed 9 years ago by wd

some cosmetics

Changed 9 years ago by dnh

mc-extfs-iso9660-xorriso.3.patch

comment:2 Changed 9 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 9 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 9 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 9 years ago by holbru

comment:5 follow-up: ↓ 6 Changed 9 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 9 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 9 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 9 years ago by slavazanko

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

comment:9 Changed 9 years ago by slavazanko

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

comment:10 Changed 9 years ago by slavazanko

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