I've found a bug in the copy-flex-dependencies goal regarding versionRange,
actually that mojo is unable to work with version ranges...

In addition that goal uses compile time dependencies but I think it should
use runtime dependencies as module swf should have a runtime scope... am i
right?

Can I fix them both?

On Mon, Jan 24, 2011 at 19:10, Roberto Lo Giacco <[email protected]>wrote:

> POM fixes and a fix for bug in the InstallerMojo found by Sebastien Le
> Callonnec have been pushed into my fork and a pull request has been
> submitted.
>
> Thank you,
>   Roberto
>
>
> On Mon, Jan 24, 2011 at 12:19, Marvin Froeder <[email protected]> wrote:
>
>> FM3 has a stick with the oldest possible policy.....
>> FM4 is the opposite... update ASAP.....
>>
>> On Mon, Jan 24, 2011 at 10:14 AM, Roberto Lo Giacco 
>> <[email protected]>wrote:
>>
>>> And finally, do you wish me to update the flex sdk version to 3.5 or do
>>> you prefer to have flexmojos 3 sticking to 3.2? Actually in my tests I've
>>> used 3.5.
>>>
>>>
>>> On Mon, Jan 24, 2011 at 12:12, Roberto Lo Giacco <[email protected]>wrote:
>>>
>>>> The two poms involved are flexmojos-testing/flexmojos-fbtest-harness
>>>> and flexmojos-testing/flexmojos-test-harness: they both refer to a
>>>> 3.8-SNAPSHOT parent which actually is 3.9-SNAPSHOT.
>>>>
>>>> This lead to a non working parent pom resolution.
>>>>
>>>>
>>>> On Mon, Jan 24, 2011 at 12:02, Roberto Lo Giacco 
>>>> <[email protected]>wrote:
>>>>
>>>>> I've seen you have pulled my changes: are you going to release the
>>>>> changes soon or do you prefer to wait a little bit? I think I've found a
>>>>> couple of errors in the poms too, as they reference the old 3.8-SNAPSHOT
>>>>> instead of the 3.9-SNAPSHOT... If you search for the `3.8-SNAPSHOT` string
>>>>> you can find them, but if you prefer I can submit another change.
>>>>>
>>>>>
>>>>> On Sun, Jan 23, 2011 at 22:22, Roberto Lo Giacco 
>>>>> <[email protected]>wrote:
>>>>>
>>>>>> I've pushed the changes to my fork, waiting for your comments.
>>>>>>
>>>>>> On Thu, Jan 20, 2011 at 13:31, Roberto Lo Giacco <[email protected]
>>>>>> > wrote:
>>>>>>
>>>>>>> Ok, I'll give a look on how much it need to be changed to fix our
>>>>>>> needs.... I'll be back to you soon on this
>>>>>>>
>>>>>>>
>>>>>>> On Thu, Jan 20, 2011 at 12:29, Marvin Froeder <[email protected]>wrote:
>>>>>>>
>>>>>>>> Well, have 2 goal with similar logic means one will evolve and the
>>>>>>>> other will be forgotten =/
>>>>>>>>
>>>>>>>> Also, CopyMojo also has logic to deal with modules, runtime
>>>>>>>> localization and rsls....  this DependencyProcessorMojo only covers 
>>>>>>>> RSLs....
>>>>>>>>
>>>>>>>>
>>>>>>>> VELO
>>>>>>>>
>>>>>>>>
>>>>>>>> On Thu, Jan 20, 2011 at 9:02 AM, Roberto Lo Giacco <
>>>>>>>> [email protected]> wrote:
>>>>>>>>
>>>>>>>>> If you prefer I can patch that goal, but I think a different goal
>>>>>>>>> better fit our needs. The CopyMojo works perfectly for war files and 
>>>>>>>>> I love
>>>>>>>>> it, patching it can lead to a lot os switches in the code bacause of
>>>>>>>>> different default output directories, different artifact ype, etc... 
>>>>>>>>> And I
>>>>>>>>> didn't want to break anything in the existing code so no complains 
>>>>>>>>> will
>>>>>>>>> raise on the new version as many people is still using that version.
>>>>>>>>>
>>>>>>>>> I just wished to be on the safe side and I thought adding a
>>>>>>>>> different mojo could be the safest solution... But I see your point 
>>>>>>>>> and I
>>>>>>>>> can refactor the CopyMojo if you wish.... still I think it's not the 
>>>>>>>>> safest
>>>>>>>>> solution.
>>>>>>>>>
>>>>>>>>> Cheers
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> On Thu, Jan 20, 2011 at 10:51, Marvin Froeder 
>>>>>>>>> <[email protected]>wrote:
>>>>>>>>>
>>>>>>>>>> I don't see the point on creating a second goal that does almost
>>>>>>>>>> the same thing....  it would make more sense patch the goal that 
>>>>>>>>>> already
>>>>>>>>>> exists then....
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On Thu, Jan 20, 2011 at 7:27 AM, Roberto Lo Giacco <
>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>
>>>>>>>>>>> Yes, it partially overlap but the CopyMojo works on war artifacts
>>>>>>>>>>> only while we wish to copy dependent artifacts into SWF project as 
>>>>>>>>>>> well for
>>>>>>>>>>> testing purposes: our flex project is modular and we want to test 
>>>>>>>>>>> it out of
>>>>>>>>>>> the scope of a war file...
>>>>>>>>>>>
>>>>>>>>>>> The alternative was to process the dependencies, including
>>>>>>>>>>> transitive ones, and copy all the artifacts using the dependency 
>>>>>>>>>>> plugin, but
>>>>>>>>>>> as we use non-standard scopes (rsl, caching) which change the 
>>>>>>>>>>> artifact type
>>>>>>>>>>> `on the fly` it was not applicable.
>>>>>>>>>>>
>>>>>>>>>>> Do you see my point or should I further explain?
>>>>>>>>>>>
>>>>>>>>>>> On Wed, Jan 19, 2011 at 23:37, Marvin Froeder <[email protected]
>>>>>>>>>>> > wrote:
>>>>>>>>>>>
>>>>>>>>>>>> What is this DependencyProcessorMojo for? Doesn't it overlay
>>>>>>>>>>>> CopyMojo?
>>>>>>>>>>>>
>>>>>>>>>>>> VELO
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> On Wed, Jan 19, 2011 at 6:33 PM, Roberto Lo Giacco <
>>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>>> Pull request made, but I think I've to process the additional
>>>>>>>>>>>>> `cache` scope into the DependencyProcessorMojo... but I'm not 
>>>>>>>>>>>>> sure...
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Wed, Jan 19, 2011 at 13:44, Marvin Froeder <velo.br@
>>>>>>>>>>>>> gmail.com> wrote:
>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Wed, Jan 19, 2011 at 11:41 AM, Roberto Lo Giacco <
>>>>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Pushing to my fork right now.
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Ok, make pull requests to my fork and I will take a look ASAP.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> VELO
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>  --
>>>>>>>>>>>>>> You received this message because you are subscribed to the
>>>>>>>>>>>>>> Google
>>>>>>>>>>>>>> Groups "Flex Mojos" group.
>>>>>>>>>>>>>> To post to this group, send email to
>>>>>>>>>>>>>> [email protected]
>>>>>>>>>>>>>> To unsubscribe from this group, send email to
>>>>>>>>>>>>>> [email protected]<flex-mojos%[email protected]>
>>>>>>>>>>>>>> For more options, visit this group at
>>>>>>>>>>>>>> http://groups.google.com/group/flex-mojos
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> http://flexmojos.sonatype.org/
>>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>  --
>>>>>>>>>>>>> You received this message because you are subscribed to the
>>>>>>>>>>>>> Google
>>>>>>>>>>>>> Groups "Flex Mojos" group.
>>>>>>>>>>>>> To post to this group, send email to
>>>>>>>>>>>>> [email protected]
>>>>>>>>>>>>> To unsubscribe from this group, send email to
>>>>>>>>>>>>> [email protected]<flex-mojos%[email protected]>
>>>>>>>>>>>>> For more options, visit this group at
>>>>>>>>>>>>> http://groups.google.com/group/flex-mojos
>>>>>>>>>>>>>
>>>>>>>>>>>>> http://flexmojos.sonatype.org/
>>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>  --
>>>>>>>>>>>> You received this message because you are subscribed to the
>>>>>>>>>>>> Google
>>>>>>>>>>>> Groups "Flex Mojos" group.
>>>>>>>>>>>> To post to this group, send email to
>>>>>>>>>>>> [email protected]
>>>>>>>>>>>> To unsubscribe from this group, send email to
>>>>>>>>>>>> [email protected]<flex-mojos%[email protected]>
>>>>>>>>>>>> For more options, visit this group at
>>>>>>>>>>>> http://groups.google.com/group/flex-mojos
>>>>>>>>>>>>
>>>>>>>>>>>> http://flexmojos.sonatype.org/
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>  --
>>>>>>>>>>> You received this message because you are subscribed to the
>>>>>>>>>>> Google
>>>>>>>>>>> Groups "Flex Mojos" group.
>>>>>>>>>>> To post to this group, send email to [email protected]
>>>>>>>>>>> To unsubscribe from this group, send email to
>>>>>>>>>>> [email protected]<flex-mojos%[email protected]>
>>>>>>>>>>> For more options, visit this group at
>>>>>>>>>>> http://groups.google.com/group/flex-mojos
>>>>>>>>>>>
>>>>>>>>>>> http://flexmojos.sonatype.org/
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>  --
>>>>>>>>>> You received this message because you are subscribed to the Google
>>>>>>>>>> Groups "Flex Mojos" group.
>>>>>>>>>> To post to this group, send email to [email protected]
>>>>>>>>>> To unsubscribe from this group, send email to
>>>>>>>>>> [email protected]<flex-mojos%[email protected]>
>>>>>>>>>> For more options, visit this group at
>>>>>>>>>> http://groups.google.com/group/flex-mojos
>>>>>>>>>>
>>>>>>>>>> http://flexmojos.sonatype.org/
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>  --
>>>>>>>>> You received this message because you are subscribed to the Google
>>>>>>>>> Groups "Flex Mojos" group.
>>>>>>>>> To post to this group, send email to [email protected]
>>>>>>>>> To unsubscribe from this group, send email to
>>>>>>>>> [email protected]<flex-mojos%[email protected]>
>>>>>>>>> For more options, visit this group at
>>>>>>>>> http://groups.google.com/group/flex-mojos
>>>>>>>>>
>>>>>>>>> http://flexmojos.sonatype.org/
>>>>>>>>>
>>>>>>>>
>>>>>>>>  --
>>>>>>>> You received this message because you are subscribed to the Google
>>>>>>>> Groups "Flex Mojos" group.
>>>>>>>> To post to this group, send email to [email protected]
>>>>>>>> To unsubscribe from this group, send email to
>>>>>>>> [email protected]<flex-mojos%[email protected]>
>>>>>>>> For more options, visit this group at
>>>>>>>> http://groups.google.com/group/flex-mojos
>>>>>>>>
>>>>>>>> http://flexmojos.sonatype.org/
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>  --
>>> You received this message because you are subscribed to the Google
>>> Groups "Flex Mojos" group.
>>> To post to this group, send email to [email protected]
>>> To unsubscribe from this group, send email to
>>> [email protected]<flex-mojos%[email protected]>
>>> For more options, visit this group at
>>> http://groups.google.com/group/flex-mojos
>>>
>>> http://flexmojos.sonatype.org/
>>>
>>
>>  --
>> You received this message because you are subscribed to the Google
>> Groups "Flex Mojos" group.
>> To post to this group, send email to [email protected]
>> To unsubscribe from this group, send email to
>> [email protected]<flex-mojos%[email protected]>
>> For more options, visit this group at
>> http://groups.google.com/group/flex-mojos
>>
>> http://flexmojos.sonatype.org/
>>
>
>

-- 
You received this message because you are subscribed to the Google
Groups "Flex Mojos" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/flex-mojos

http://flexmojos.sonatype.org/

Reply via email to