make appOutputDir a full path about gradle-macappbundle

[GoogleCodeExporter]

The gradle convention seems to be that "output" paths are relative to the build 
dir and inputs are relative to the project dir, so that is what I have 
followed. Note that not specifying a "prefix" to the path, ie making it look 
like a full path, defaults to being relative to the project dir.

I have updated the comments on the appOutputDir and dmgOutputDir to say 
relative to build directory.

I think this convention makes sense, inputs in project directory, outputs in 
build directory.

Original comment by [email protected] on 5 Oct 2012 at 1:42

• Changed state: WontFix

[GoogleCodeExporter]

I've seen lots of plugins which pass a "file" instead of a string path for 
inputs. That seems more robust to me (since it will fail in a clear place).

If we keep your approach, then what about the icon and background images? 
Should they also be relative paths?

Original comment by [email protected] on 8 Oct 2012 at 2:23

[GoogleCodeExporter]

From the gradle api docs, the Project.file method:
   Resolves a file path relative to the project directory of this project.
so they are relative, just relative to the project directory instead of the 
build directory.

http://gradle.org/docs/current/dsl/org.gradle.api.Project.html#org.gradle.api.Pr
oject:file%28java.lang.Object%29

Original comment by [email protected] on 8 Oct 2012 at 11:55

[GoogleCodeExporter]

Yes, correct. But the resolution is in the build.gradle, not inside the plugin. 
This means that the user has complete control over what they pass.

I don't think this matters too much, but the inconsistency between sometimes 
passing paths and sometimes path fragments is confusing.

Original comment by [email protected] on 10 Oct 2012 at 1:45

[GoogleCodeExporter]

Humm, I suppose I could use File instead of String, set the defaults as they 
are now and overload the setter so that a string does the resolution relative 
to the corresponding directory. 

Changing back to Accepted.

Original comment by [email protected] on 10 Oct 2012 at 12:20

• Changed state: Accepted

[GoogleCodeExporter]

Original comment by [email protected] on 10 Oct 2012 at 12:20