lukeme / gobible

Automatically exported from code.google.com/p/gobible
1 stars 0 forks source link

Patch to fix file separator in GoBibleCreator.java #165

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Greg Hellings has reported an issue arising from his attempt to use Go Bible 
Creator in a Linux system.

The offending code is line 696 of GoBibleCreator.java which reads: 

String sFilename = folder.toString() + "\\" + listOfFiles[i].getName();

Observe the Windows style file separator "\\" is the cause of the issue.

It should read:

String sFilename = folder.toString() + File.separator + 
listOfFiles[i].getName();

Being a programmer himself, he's investigated the problem and written a patch 
to fix it. This is attached.

After recompiling Go Bible Creator with the patch applied, he was successful in 
using it to make a Go Bible application with USFM files as source text.

Original issue reported on code.google.com by DFH...@gmail.com on 11 Aug 2012 at 4:41

Attachments:

GoogleCodeExporter commented 8 years ago
It's conceivable that a similar patch might be required for the SymScroll 
branch.

Issue assigned to Daniel.

The version number of Go Bible Creator should be incremented after applying the 
patch.

Original comment by DFH...@gmail.com on 11 Aug 2012 at 4:44

GoogleCodeExporter commented 8 years ago

Original comment by DFH...@gmail.com on 14 Dec 2012 at 2:05

GoogleCodeExporter commented 8 years ago
Issue 167 has been merged into this issue.

Original comment by DFH...@gmail.com on 14 Dec 2012 at 2:05