[Bug 838720] Re: Crash on folder name with spaces
** Changed in: iso-scan (Debian) Status: New => Fix Released -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 838720] Re: Crash on folder name with spaces
As a followup, after installing Precise beta 2 with iso-scan 1.37ubuntu1, I am still affected by this bug. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 838720] Re: Crash on folder name with spaces
The attachment "iso-scan.postinst.diff" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors please also unsubscribe the team from this bug report. [This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.] ** Tags added: patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 838720] Re: Crash on folder name with spaces
** Changed in: iso-scan (Debian) Status: Unknown => New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 838720] Re: Crash on folder name with spaces
** Bug watch added: Debian Bug tracker #640789 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=640789 ** Also affects: iso-scan (Debian) via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=640789 Importance: Unknown Status: Unknown -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 838720] Re: Crash on folder name with spaces
Sylvain, this package is small so your diff shouldn't be a problem but when using diff directly its a good practice that all +++ lines, which are like metadata about where to apply the patch, contain the same name as the originals with a relative path 1 level higher than the package tree, specially when the diff patchesmore than one file. So it gets like this: --- iso-scan.orig/debian/iso-scan.postinst +++ iso-scan/debian/iso-scan.postinst And the dev, just needs to cd $PACKAGE; patch -p1 < patchfile instead of patch originalfile patchfile I'm not sure why I had doubts in #1 but this bug does affect debian too and its just that line (I thought there were 2...). I'll open a bug upstream. Optionally, If you want, you can fix it directly by purposing a merge to the branch lp:~ubuntu-core-dev/iso-scan/ubuntu using the procedure mentioned in https://wiki.ubuntu.com/Bugs/HowToFix -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 838720] Re: Crash on folder name with spaces
This is the result of a diff command between iso-scan.postinst and my proposed_iso-scan.postinst files Thanks Sérgio ** Patch added: "iso-scan.postinst.diff" https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+attachment/2340116/+files/iso-scan.postinst.diff -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 838720] Re: Crash on folder name with spaces
Thanks for reporting this bug and any supporting documentation. Since this bug has enough information provided for a developer to begin work, I'm going to mark it as confirmed and let them handle it from here. Thanks for taking the time to make Ubuntu better! To create a patch look here[1] or you can always use diff -u. However I don't know where you should apply the patch. I took a look at debian's iso-scan and seems to me it also suffers from similar unquoted $dir's. [1]: https://wiki.ubuntu.com/PackagingGuide/Complete#Authoring_a_patch ** Changed in: iso-scan (Ubuntu) Status: New => Confirmed -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/838720 Title: Crash on folder name with spaces To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/iso-scan/+bug/838720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs