search

annotationsatharvard / catcha

Catch-A - Catching Annotation: An annotation backend and API.
http://catcha.readthedocs.io
20 stars 9 forks source link

Remove duplicate Catch-config.properties file #87

Closed jmiranda closed 7 years ago

jmiranda commented 7 years ago

Question from @nmaekawa

I started poking at the catcha repo and I having trouble with the “catch-config.properties” file. From a fresh git clone it already shows that it’s modified and doesn’t let me change to another branch if I don’t commit the changes – even though I didn’t even touch the file!

Paolo (the original developer) originally added that property file (Catch-config.properties) to the root of the project. I think I clumsily added the catch-config.properties since I changed the appName to catch. However, I don't use either of those files. Instead, I prefer to keep the properties file outside of the project and under my home directory (/home/jmiranda/.grails/catch-config.properties).

For Mac, you can probably move the catch-config.properties file under /Users//.grails/catch-config.properties. Then you can just revert any changes you made to catch-config.properties in the project root to avoid any git problems - not sure what would have changed those files.

In addition, I'll add those files to .gitignore (to avoid the git issues) and rewrite the dev setup instructions to say "copy the file from the project root to you ${userHome}/.grails/catch-config.properties".

Seems like I need to remove the Catch-config.properties file as well.

jmiranda commented 7 years ago

Another comment from @nmaekawa

I think there is a “Catch-config.properties” and “catch-config.properties”, and git somehow confuses both because they have the same content. Do you experience the same?

Yes, it appears there are two files - I'll clean that up. I wonder if Mac is having trouble because the filesystem is case-insensitive. You have edited the file, but the Mac probably only knows about one of the files since it wasn't able to create both due to the case insensitivity issue. Let me know if that problem continues and I'll delete Catch-config.properties.