[BitPim-devel] [bitpim-cvs-checkins] SF.net SVN: bitpim: [3322] tr unk/bitpim/src

[Joe Pham]

This is a really bad change to make. Our code has to work with case
sensitive operating systems and assumes lower case everywhere. This
change will cause random failures for users on Linux and Mac
depending on filesystem type.

I had thought 'bout the Linux & Mac platforms, but I figure the worst case is that BitPim would pickup files that may not have been intended for the build. On Windows, the discrepancy is that it'd work fine in the debug mode, but not in the build. We'd just need to keep those extensions in lower case.

-Joe Pham



_____________________________________________________________________
Call Anyone, Anytime, Anywhere in the World - FREE!
Free Internet calling from NetZero Voice
Visit http://www.netzerovoice.com today!

[Roger Binns]

Post by Joe Pham
I had thought 'bout the Linux & Mac platforms, but I figure the worst case
is that BitPim would pickup files that may not have been intended for the build.

No, the build wouldn't run. For example the code may reference 'foo.png' but
it is stored as 'foo.PNG' on the filesystem. Windows will run fine, but on
Linux the file won't be packaged or found at run time.

Post by Joe Pham
We'd just need to keep those extensions in lower case.

I agree. But the SVN change is unnecessary and it will make the Windows
build unexpectedly work while case sensitive platforms won't. Our build/
release tool shouldn't make the platforms be gratiously different and
hence harder to troubleshoot.

Roger