Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

isoinfo adds ";1" to the end of file name when Joliet without Rock Ridge is used #2471

Closed
mc-butler opened this issue Jan 11, 2011 · 9 comments
Assignees
Labels
area: vfs Virtual File System support prio: low Minor problem or easily worked around ver: 4.7.5 Reproducible in version 4.7.5
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2471
Reporter asy (a_s_y@….ru)

$ isoinfo -d -i tc1000_doc_lib_v.4.3.iso

CD-ROM is in ISO 9660 format
System id: CD-RTOS CD-BRIDGE
Volume id: 10043109
Volume set id:
Publisher id:
Data preparer id:
Application id: EASY CD CREATOR 5.0 (306) COPYRIGHT (C) 1999-2001 ROXIO, INC.
Copyright File id:
Abstract File id:
Bibliographic File id:
Volume set size is: 0
Volume set sequence number is: 0
Logical block size is: 2048
Volume size is: 43970
Joliet with UCS level 3 found
NO Rock Ridge present

$ isoinfo -J -l -i tc1000_doc_lib_v.4.3.iso

Directory listing of /
d---------   0    0    0             556 Jun 14 2001 [     21 02]  .
d---------   0    0    0             556 Jun 14 2001 [     21 02]  ..
d---------   0    0    0            1204 Jun 14 2001 [     22 02]  html
d---------   0    0    0             622 Jun 14 2001 [     23 02]  index
----------   0    0    0             950 Jun 13 2001 [     87 00]  index.pdx;1
d---------   0    0    0             214 Jun 14 2001 [     24 02]  install
----------   0    0    0           24192 Jun 14 2001 [    164 00]  mainmenu.gif;1
----------   0    0    0            2711 Jun 14 2001 [    108 00]  mainmenu.htm;1
d---------   0    0    0            3488 Jun 14 2001 [     25 02]  pdfdocs


The patch in attachment.

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by asy (a_s_y@….ru) on Jan 11, 2011 at 14:12 UTC

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 15, 2011 at 15:34 UTC (comment 1)

Can we get rid of isoinfo -d -i double run in test_iso()?

@mc-butler
Copy link
Author

Changed by asy (a_s_y@….ru) on Jan 16, 2011 at 9:14 UTC (comment 2)

It is possible but it complicate code. Need something like

ISOINFO_D_I=`isoinfo -d -i "$1"`

if [ "Joliet with UCS level 3 found" == "`echo $ISOINFO_D_I|sed "s/.*\(Joliet with UCS level 3 found\).*/\1/"`" ]; then
 echo 1
fi

On the other hand "isoinfo-d-i" is cached and works fast second and more time:

$ time isoinfo -d -i tc1000_doc_lib_v.4.3.iso | tail
0.00user 0.00system 0:00.00elapsed 400%CPU (0avgtext+0avgdata 2672maxresident)k
0inputs+0outputs (0major+249minor)pagefaults 0swaps

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 4, 2011 at 7:39 UTC (comment 3)

  • Owner set to andrew_b
  • Status changed from new to accepted
  • Milestone changed from 4.8 to 4.8.0-pre1
  • Keywords iso9660 deleted

Created 2471_iso9660_semicolon branch. Parent branch is 4.7.5-stable.
Initial [29c5beb766125c78170971e19515bd5abe7f820f]

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 4, 2011 at 7:42 UTC (comment 4)

  • Severity changed from no branch to on review

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Feb 4, 2011 at 11:18 UTC (comment 5)

  • Votes set to slavazanko

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Feb 4, 2011 at 13:03 UTC (comment 6)

  • Votes changed from slavazanko to slavazanko angel_il
  • Severity changed from on review to approved

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 4, 2011 at 16:29 UTC (comment 7)

  • Resolution set to fixed
  • Votes changed from slavazanko angel_il to committed-master committed-stable
  • Status changed from accepted to testing
  • Severity changed from approved to merged

Merged to 4.7.5-stable.
[876b167]

Cherry-picked to master.
[3293c5b]

Cherry-picked to 4.7.0-stable.
[769a1a10b05afdc78d5e91b0220856dfec52d9a7]

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 4, 2011 at 16:33 UTC (comment 8)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: vfs Virtual File System support prio: low Minor problem or easily worked around ver: 4.7.5 Reproducible in version 4.7.5
Development

No branches or pull requests

2 participants