Skip to content
This repository was archived by the owner on Oct 23, 2024. It is now read-only.

Conversation

@craftoid
Copy link

@craftoid craftoid commented Feb 5, 2013

Problem

When several people work on a Processing library or tool via git, they usually have different classpaths, sketchbook locations etc. These things are configured in a file that is currently under version control (which is a bad thing).

Solution

  • Create custom config files named local.properties that are excluded from version control.
  • Provide template files: local.properties.tmpl that the developers need to copy and customize to their needs.
  • modify the build process to use both local.properties and build.properties

For all three templates the build process was modified,
so that local configuration for the build system can be seperated
from the project configuration.

This is required to allow several people to work on a library
or tool via version control.

Local configurations of the build system are now supposed to reside
in 'local.properties', a file which is excluded from revision control

Each template now has a 'local.properties.tmpl', that each developer
needs to customize and copy to 'local.properties'

For each of the three templates the following steps have been done:

 * create local.properties.tmpl
 * hide local.properties and distribution directory from revision control
 * update build process to parse both local.properties and build.properties
 * update changelog
@kritzikratzi
Copy link

couldn't you just add this first line

<property file="./resources/local.properties" />

to the top of your build.xml to achieve the same?
i didn't check, but it even shouldn't matter if the file doesn't exist.

@kritzikratzi
Copy link

ah, sorry, i just realize thats exactly what you did. why all the other changes then? looks like a big change for an easy problem :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants