MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004361ardourfeaturespublic2011-10-09 11:122015-09-18 08:19
Reportercolinf 
Assigned Topaul 
PrioritynormalSeveritytweakReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version2.8.13Fixed in Version 
Summary0004361: [PATCH] Don't default to or allow colons in snapshot name
DescriptionI occasionally have to back up Ardour session directories to FAT file systems. Mostly this is fine, but FAT doesn't really get along well with file names containing colons, and the default snapshot name contains not one, but two of them.
Additional InformationAttached a patch for A3 to replace the colons with full stops in the default snapshot name, and to disallow colons, exactly as forward-slash and backslash characters are already forbidden.

I realise that the resulting time string no longer conforms to ISO 8601, but at least I can copy the files to a FAT-formatted drive.
TagsNo tags attached.
Attached Filespatch file icon no-colons-in-snapshot-names.A3.patch [^] (772 bytes) 2011-10-09 11:12 [Show Content]
patch file icon no-colons-in-snapshot-names.A2.patch [^] (788 bytes) 2011-10-09 11:12 [Show Content]

- Relationships

-  Notes
(0011674)
colinf (updater)
2011-10-09 11:13

And for the sake of completeness, here's a patch against A2 as well.
(0011676)
danboid (reporter)
2011-10-10 16:37

Good spot and thanks for fixing this Colin (presuming your patch does actually work of course)! I bet a few people have fallen foul to this and not known why.
(0011697)
cth103 (administrator)
2011-10-17 13:36

Applied to A3 in SVN 10213. Thanks!
(0013243)
paul (administrator)
2012-05-15 05:19

applied to 2.0-ongoing as rev 12228. thanks!
(0017301)
colinf (updater)
2015-09-18 08:19

Closing old issues reported by me: these have long since been fixed.

- Issue History
Date Modified Username Field Change
2011-10-09 11:12 colinf New Issue
2011-10-09 11:12 colinf File Added: no-colons-in-snapshot-names.A3.patch
2011-10-09 11:12 colinf File Added: no-colons-in-snapshot-names.A2.patch
2011-10-09 11:13 colinf Note Added: 0011674
2011-10-10 16:17 cth103 cost => 0.00
2011-10-10 16:17 cth103 Target Version => 3.0-beta1
2011-10-10 16:17 cth103 Summary Don't default to or allow colons in snapshot name => [PATCH] Don't default to or allow colons in snapshot name
2011-10-10 16:37 danboid Note Added: 0011676
2011-10-17 13:36 cth103 Note Added: 0011697
2011-10-17 13:36 cth103 Target Version 3.0-beta1 => 2.8.13
2012-05-15 05:19 paul Note Added: 0013243
2012-05-15 05:19 paul Status new => resolved
2012-05-15 05:19 paul Resolution open => fixed
2012-05-15 05:19 paul Assigned To => paul
2015-09-18 08:19 colinf Note Added: 0017301
2015-09-18 08:19 colinf Status resolved => closed


Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker