Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Making Processing Library Templates Fit For Collaboration #1

Open
craftoid opened this issue Aug 17, 2013 · 2 comments
Open

Making Processing Library Templates Fit For Collaboration #1

craftoid opened this issue Aug 17, 2013 · 2 comments

Comments

@craftoid
Copy link

Problem

When several people work on a Processing library or tool via git, they usually have different operating systems, classpaths, sketchbook locations etc. These things are configured in build.properties, and are 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

Implementation

I already implemented a solution and opened a pull-request for the old repo, which you can find here:
processing/processing-templates#1

@prisonerjohn
Copy link
Member

Hi Martin,

First of all, sorry for not noticing this pull request on the old repo, it must've slipped through the cracks.

Thanks for this, these updates are very useful for teams working on a library or tool. However, most contributions to Processing are developed by individuals. My goal with the library template is to have an as easy as possible starting point to offer developers, which they can modify to suit their needs.

As it stands, the template user only needs to edit a single file to configure their project. This config file feeds the Ant script, library properties file, and web directory. I feel that the extra steps involved with your changes (duplicating and renaming a second properties file, editing settings in both local.properties and build.properties) will not be advantageous to most users, who are working on their own.

Maybe the new git-friendly templates will encourage collaboration, in which case we should definitely revisit a solution such as this one!

@craftoid
Copy link
Author

craftoid commented Sep 7, 2013

Hi Elie,

Thank you for your response!
The suggested changes emerged in a a situation where I started to work on a tool collaboratively.

I must say, I totally agree with you:
Simpilicity first!

However:
The problem with the simple solution is that even if some folks just wants to pull
the repo, and build the library on their own machine, they always have to manually
adjust the build.properties file every single time they issue a pull request.

Probably it would be best to have different git branches, each with a different pre-configured properties file.
Like one for OSX, one for Windows, one for Linux and one for collaborative work.
So if you wanted to start a new library, you would just fork the branch that suits your needs,
and it would work out of the box for most users...

prisonerjohn pushed a commit that referenced this issue Nov 25, 2016
build.properties to Processing 3.2.3 and Java 8
jeremydouglass referenced this issue in archive-for-processing/processing-forked-libraries Apr 11, 2020
jeremydouglass referenced this issue in archive-for-processing/processing-forked-libraries Apr 11, 2020
getting rid of local dependencies
jeremydouglass referenced this issue in archive-for-processing/processing-forked-libraries Apr 11, 2020
Merge Forces into Master
jeremydouglass referenced this issue in archive-for-processing/processing-forked-libraries Apr 11, 2020
Renamed 'binaries' to 'library'
jeremydouglass referenced this issue in archive-for-processing/processing-forked-libraries Apr 11, 2020
jeremydouglass referenced this issue in archive-for-processing/processing-forked-libraries Apr 11, 2020
Add a Gitter chat badge to README.md
jeremydouglass referenced this issue in archive-for-processing/processing-forked-libraries Apr 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants