| Summary: | Failsafe EOL conversion | ||
|---|---|---|---|
| Product: | Tomcat 5 | Reporter: | Sebb <sebb> |
| Component: | Native:Packaging | Assignee: | Tomcat Developers Mailing List <dev> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | P2 | ||
| Version: | 5.5.27 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows XP | ||
| Attachments: |
List of extensions used in 5.5.28
List of files with no extension fixcrlf use include rather than exclude Improved patch - detab |
||
Created attachment 24067 [details]
List of files with no extension
So the only safe way to do it, would be to specify an includes list instead of using an excludes. Patches welcome OK, I will create a patch shortly Created attachment 24080 [details]
fixcrlf use include rather than exclude
Created attachment 24081 [details]
Improved patch - detab
Modified patch applied to trunk and proposed for 6.0.x and 5.5.x Fixed in 6.0.22, 5.5.29. |
Created attachment 24066 [details] List of extensions used in 5.5.28 The builds/build.xml file has the following code: <target name="package-src-tgz"> <fixcrlf srcdir="${tomcat.dist}/src" excludes="**/*.bin,**/*.bmp,**/*.dia,**/*.exe,**/*.gif,**/*.ico,**/*.jar,**/*.jpg,**/*.pdf,**/*.rtf,**/*.war" eol="lf" encoding="ISO-8859-1" fixlast="false" /> The intention is to fix all the source files so that they are readable on Unix (LF) systems. However, the command is inherently unsafe - if a new binary file is added with an extension that is not listed, then the file will be damaged. In general it is impossible to reverse LF-conversion on a binary file. It would be much safer to use includes instead. If a new source file type is added, then the worst that can happen is that the file is awkard to read on Unix. Unlike the case with binary files, the file can still be converted successfully. The current list of extensions that are present in the 5.5.28 source archive are included in the file "extensions.txt". At least two binary extensions (.db, .keystore) are currently missing from the excludes list. There are quite a few files with no extension. These appear to be all text files, and are listed in the file "no-extension.txt" (duplicate names have been removed).