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

build documentation (on wiki) is out of date #1629

Closed
ybakos opened this issue Feb 15, 2013 · 28 comments
Closed

build documentation (on wiki) is out of date #1629

ybakos opened this issue Feb 15, 2013 · 28 comments

Comments

@ybakos
Copy link
Contributor

ybakos commented Feb 15, 2013

https://github.com/processing/processing/wiki/Build-Instructions

This is an issue for discussing updates to the build doc, which is something I'm actively working on.

@prisonerjohn: Elie, let me know if you're already on this and/or we can collaborate on parts.

Let's make this doc "real nice".

@benfry
Copy link
Contributor

benfry commented Feb 15, 2013

I've fixed the Eclipse section, though it could use more detail/screenshots to explain how to get the beast running. But at least it's not longer wildly inaccurate.

The rest of the command line stuff needs to be explained in more detail. It's much simpler than it all looks, but we do need better instructions.

@prisonerjohn
Copy link
Member

Yeah I was planning on updating it. First step was to move everything over, which I did yesterday. We can definitely collaborate on this.

The good news is that it looks like the process is a lot simpler, just running ant in the build directory.

@ybakos I'm on OS X, if you're on Windows or Linux, maybe we could split the work that way.

@ybakos
Copy link
Contributor Author

ybakos commented Feb 15, 2013

Ok. I'd like to casually suggest the goal of the build instructions to be cd build and ant.

I've confirmed that this works on OSX 10.7.3. I'll work on linux and windows today.

@ybakos
Copy link
Contributor Author

ybakos commented Feb 20, 2013

No issues on WIn7/8. Testing linux (Ubuntu) now.

I feel that the document should explain pre-requisites (java7, ant, paths correctly set up) but not describe steps to getting those things set up.

@Manindra29
Copy link
Member

I've made an effort here: https://github.com/processing/processing/wiki/Build-Instructions-(Updated)
Comments and edits welcome.

@ybakos
Copy link
Contributor Author

ybakos commented Feb 22, 2013

@benfry @prisonerjohn What do you think about starting with this: https://gist.github.com/ybakos/5008910

We can add visual aids to the Eclipse section later.

@ybakos
Copy link
Contributor Author

ybakos commented Feb 25, 2013

Perhaps we shouldn't use the github wiki at all? That way all Processing doco is in one place, on wiki.processing.org.

Maybe place bare-minimum build instructions in the README.

@REAS
Copy link
Member

REAS commented Feb 25, 2013

@ybakos For now, we want to use both. All development information (tools, library, and main project) should be on the GitHub wikis and the Processing wiki should have clear links to GitHub. We need the GitHub wiki to help developers find the information they need easily and we need the Processing wiki for additional information, not related to development. (We're currently talking about phasing out the Processing wiki.)

@joshgiesbrecht
Copy link
Contributor

Would like to add for the Eclipse section, to actually run the IDE in Eclipse for debugging I had to change the working directory to \build\windows\work for it to run. Not sure if that's common for everyone using Eclipse on Windows but might be worth adding. (Took me a couple hours to hunt down where it was expecting to look.)

@benfry
Copy link
Contributor

benfry commented Jun 3, 2013

Yep, when running from Eclipse (for any platform) the working directory needs to be 'work'. Using Eclipse is simply not documented at the moment. I use it for development, but we're not committed to that being a development path or that always working out of the box.

@joshgiesbrecht
Copy link
Contributor

Ok. I'll add that as a note to the Eclipse section of the build page. (Every little bit helps, hopefully)

@REAS
Copy link
Member

REAS commented Jun 3, 2013

@joshgiesbrecht Yes, help to keep this all updated is very appreciated. The Wiki is open for the community to contribute. If you don't have an account there yet, there are instructions on the first page.

@datguy
Copy link
Contributor

datguy commented Jun 21, 2013

@joshgiesbrecht , @benfry - setting the working directory to 'work' doesn't work on my Mac. I get an error message ("Could not read default settings. You'll need to reinstall Processing." followed by

java.io.FileNotFoundException: /Users/datguy/git/processing/build/macosx/work/lib/preferences.txt (No such file or directory)

However, if I set the working directory to /Users/datguy/git/processing/build/macosx/work/Processing.app/Contents/Resources/Java (after building with ant) it works. What am I doing wrong?

@joshgiesbrecht
Copy link
Contributor

Since my name's there, I'll just pop in to say ... I don't really know. :) I haven't used Eclipse that much, I was just happy to get it to work for myself!

@datguy
Copy link
Contributor

datguy commented Jul 18, 2013

I've been looking at the 3 versions of Build-Instructions (the current page, plus the ones proposed by @Manindra29 and @ybakos). They all say you need Java 1.7, but most of compile steps in the build.xml files specify 1.6. Is 1.7 really required, or should we change the wiki to be consistent with the build scripts?

@Manindra29
Copy link
Member

AFAIK as long as the java source and target versions are specified in the build.xml(Processing uses 1.6), compiling with even Java 1.7 produces output compatible with Java 1.6. So w.r.t building Processing, having the latest version of Java shouldn't be an issue. Not sure if this applies to OS X though.

@datguy
Copy link
Contributor

datguy commented Jul 18, 2013

I agree that the output should be compatible, but technically it's inaccurate to say that 1.7 is required if we're specifying 1.6 compatibility. I suggest that the build page(s) specify 1.6 or later until 1.7 is actually required.

@ybakos
Copy link
Contributor Author

ybakos commented Jul 18, 2013

I updated the build doc, thanks @datguy.

@dhowe
Copy link
Contributor

dhowe commented Oct 13, 2013

Not sure if this is the correct place for this question, but I was able to build OK with ant and in Eclipse on OSX 10.6.8. I was able to solve the preferences.txt issue above with a sym-link, but now I'm getting another error (when I run in Eclipse) that I'm not sure about: "FileNotFoundException: /Users/dhowe/git/processing/app/modes/java/keywords.txt"

Seems maybe the modes haven't been built properly... though they are present in path/to/processing/build/macosx/work/Processing.app/Contents/Java/modes

Any thoughts?

@benfry
Copy link
Contributor

benfry commented Oct 13, 2013

@dhowe please use the forum to ask questions. This is an issue report about the need to update the build instructions.

@datguy
Copy link
Contributor

datguy commented Oct 13, 2013

@benfry unless things have changed in the new forum, there isn't a good place there to ask questions about building or contributing to Processing.

@benfry
Copy link
Contributor

benfry commented Oct 13, 2013

@datguy Then start one. But filling a bug report about fixing the build instructions is not the place for random questions about the builds.

@REAS
Copy link
Member

REAS commented Oct 14, 2013

The new forum categories have a section called Developing Processing:
http://forum.processing.org/two/categories/developing-processing

@benfry
Copy link
Contributor

benfry commented Oct 20, 2013

Build instructions now updated, and also include the changes for 2.1.

@benfry benfry closed this as completed Oct 20, 2013
@ybakos
Copy link
Contributor Author

ybakos commented Oct 20, 2013

Shouldn't that first bullet be "install the latest JDK 7 from Oracle" instead of "JRE"?

@benfry
Copy link
Contributor

benfry commented Oct 20, 2013

No, read the rest of it.

@GKFX
Copy link
Contributor

GKFX commented Dec 1, 2013

You reference this as current from README.md but closed it last month? Should README.md be updated?

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants