Re: [Geoserver-devel] 1.3 build.xml changes

[Chris H. <ch...@op...>]

Ok, some feedback.

'ant test' is broken for me, as 'ant run' is removed.  From what I can 
tell 'run jetty' replaces 'run', and 'deploy - full' seems to replace 
the first half of 'test'.  In the description of 'deploy - full' it 
indicates that it runs jetty, but for me it definitely did not.  I'm 
fine if 'deploy - full' replaces 'test', but it should actually do what 
it says and run jetty.  And there should be a way to just build it, but 
I'll assume that's there in some form.

Also naming them 'deploy - full' with spaces is annoying, since when you 
run from the command line you now have to quote ("ant 'deploy - full'). 
  'deploy-full' is better, since I can just write 'ant deploy-full'.

There may be more, this is just my initial feedback.

Chris

Brent Owens wrote:
> Hello everyone,
> 
> I have made some changes to the 1.3 build.xml file to reflect the 
> data_dir structure.
> Before we were pulling data from the cite folders (confUserBasic etc...) 
> but now I have ported those folders over to the new data_dir folders, 
> and build the releases from them. The same data is there, but they are 
> structured differently. Each data_dir folder can be used on its own as a 
> data directory and the build file will be able to dig through the folder 
> and grab the appropriate files to make a release.
> I have also cleaned up the build.xml by removing deprecated tasks and 
> removing unused variables, renamed several tasks, and moved some tasks 
> around to better group them.
> 
> Changes:
> * cite folder: deprecated and not used in the build process
> * data_dir folder: populated with ported over cite folders, now used for 
> build process
> * build.xml:
>    - now builds from data_dir folders
>    - can still specify what config to build from (userBasic, min ... 
> using build.properties)
>    - 'geoserver.move_jar' task renamed to: 'deploy - jar'
>    - 'debug from ide' task renamed to: 'deploy - all'
>    - 'deploy - jsp' created
>    - most scripts that "create" something (a jar or whatever) are now 
> prefixed with 'build -'
>    - some more comments
> 
> (I will remove the cite folders once I am sure that the data_dir folders 
> are sufficient)
> 
> Can you guys give it a little run through for me and tell me if anything 
> is broken in the file? Thanks a bunch,
> 

-- 
Chris Holmes
The Open Planning Project
http://topp.openplans.org