Re: Issue with run-install-readers

[Jacques Le Roux <ja...@les7arts.com>]

Actually it was just a matter of removing spaces in readers names. I should have put spaces after the commas and not noticed
Anyway, I have committed a slight improvement to prevent that at r1094600 in trunk

FYI actually putting double-quotes around the readers part is also mandatory on Debian when having white spaces after comma, like 
ant run-install-readers "-Ddata-readers=seed, seed-initial, ext". It's mandatory in all cases on Windows

Jacques

From: "Jacques Le Roux" <ja...@les7arts.com>
> Actually this is not enough, it still does not work. Only the 1st reader is taken into account. 
> For run-install-readers "-Ddata-readers=seed, seed-initial, ext" you get somethink like
> 
> EntityDataLoader.java:104:INFO ] Could not find entity-data-reader named:  seed-initial. Creating a new reader with this name.
> EntityDataLoader.java:104:INFO ] Could not find entity-data-reader named:  ext. Creating a new reader with this name.
> 
> I continue to investigate...
> 
> Jacques
> 
> From: "Jacques Le Roux" <ja...@les7arts.com>
>> OK, It needs double quote on Windows XP (at least), works without on Debian
>> I put a comment at r1086642 
>> 
>> Jacques
>> 
>> From: "Adrian Crum" <ad...@sandglass-software.com>
>>> Try going back through the commit logs and see if any changes were made 
>>> to ant. It seems to think the comma is separating a list of targets to run.
>>> 
>>> -Adrian
>>> 
>>> On 3/22/2011 10:23 PM, Jacques Le Roux wrote:
>>>> Not sure maybe related to macro.xml introduction, though I don't see  
>>>> any reason reading macros.xlk code? I have tried a debug, a lot of 
>>>> lines, andn o clear result yet.
>>>> Just before entering I have
>>>>
>>>> Execute:Java13CommandLauncher: Executing 'C:\Program 
>>>> Files\Java\jdk1.6.0_22\jre\bin\java.exe' with arguments:
>>>> '-Xms128M'
>>>> '-Xmx512M'
>>>> '-XX:MaxPermSize=128m'
>>>> '-jar'
>>>> 'D:\WorkspaceNew\sparta\ofbiz.jar'
>>>> 'install'
>>>> 'readers=seed'
>>>>
>>>> The ' characters around the executable and arguments are not part of 
>>>> the command.
>>>>
>>>> And seed works well. But at the end I have
>>>>
>>>> BUILD FAILED
>>>> Target "ext" does not exist in the project "OFBiz Main Build".
>>>>        at org.apache.tools.ant.Project.tsort(Project.java:1850)
>>>>        at org.apache.tools.ant.Project.topoSort(Project.java:1758)
>>>>        at org.apache.tools.ant.Project.topoSort(Project.java:1721)
>>>>        at org.apache.tools.ant.Project.executeTarget(Project.java:1306)
>>>>        at 
>>>> org.apache.tools.ant.helper.DefaultExecutor.executeTargets(DefaultExecutor.java:41)
>>>>        at org.apache.tools.ant.Project.executeTargets(Project.java:1189)
>>>>        at org.apache.tools.ant.Main.runBuild(Main.java:758)
>>>>        at org.apache.tools.ant.Main.startAnt(Main.java:217)
>>>>        at org.apache.tools.ant.launch.Launcher.run(Launcher.java:257)
>>>>        at org.apache.tools.ant.launch.Launcher.main(Launcher.java:104)
>>>>
>>>>
>>>>
>>>> Jacques
>>>> Adrian Crum wrote:
>>>>> When was the last time it worked?
>>>>>
>>>>> -Adrian
>>>>>
>>>>> On 3/22/2011 11:52 AM, Jacques Le Roux wrote:
>>>>>> Hi,
>>>>>>
>>>>>> I think we have currently an issue with run-install-readers:
>>>>>>
>>>>>>
>>>>>> BUILD FAILED
>>>>>> Target "demo" does not exist in the project "OFBiz Main Build".
>>>>>>
>>>>>> Total time: 21 seconds
>>>>>> D:\workspace\ofbizDev>ant run-install-readers -Ddata-readers=seed,demo
>>>>>>
>>>>>> Any ideas? Do I miss something?
>>>>>>
>>>>>> Thanks
>>>>>>
>>>>>> Jacques
>>>>
>>
>