[jbossdeveloper] <repository /> on quickstarts pom.xml

Max Rydahl Andersen max.andersen at redhat.com
Tue Aug 26 06:11:06 EDT 2014


On 26 Aug 2014, at 11:27, Pete Muir wrote:

> On 26 Aug 2014, at 10:26, Paul Robinson <paul.robinson at redhat.com> 
> wrote:
>
>>
>> On 26 Aug 2014, at 10:18, Pete Muir <pmuir at redhat.com> wrote:
>>
>>>
>>> On 26 Aug 2014, at 09:27, Paul Robinson <paul.robinson at redhat.com> 
>>> wrote:
>>>
>>>> Rafael,
>>>>
>>>> It'd be worth reading the "[Wolf] Revisting embedding repositories 
>>>> in POMs for developer materials" thread on 
>>>> enterprise-maven-list at redhat.com. There was quite a bit of 
>>>> discussion about doing this. The general opinion was that it would 
>>>> be a huge win for usability, but there are quite a few subtle 
>>>> problems with the approach. For example, productization are 
>>>> currently stripping off these repository definitions from 
>>>> quickstarts, so any solution is going to need to work for them.
>>>
>>> We don’t currently put any repo definitions in quickstarts...
>>
>> Sorry, I should have been clearer. Productization would remove them 
>> if they exist. Quote from Julian Coleman:
>>
>> 	Right.  Note, also, that the vman tool will strip out the 
>> <repositories>
>> 	from the POM files that it processes.  This means that all POM files 
>> for
>> 	the IP builds (FSW/DV/BPMS) will not contain repositories.  So, 
>> adding
>> 	this information to settings.xml seems the only sensible option.
>
> Not all platforms use vman afaik.

But those that do should look into not having vman treat 
quickstarts/examples the same as the productized runtime pom's IMO - 
they aren't possible to treat the same in all situations.

If their issue is they want to control where maven get these from they 
can add it to their settings.xml proxy mappings for the repository id 
and map to their temporary test location and they can test wether the 
repository are correct.

something like:
  <mirrors>
     <mirror>
       <id>public</id>
       <mirrorOf>redhat-techpreview</mirrorOf>
       <url>file:///Users/max/testingrepo</url>
     </mirror>
   </mirrors>

/max

>>
>>
>>>
>>>>
>>>> Pete came up with an alternative approach that helps to educate the 
>>>> developer about the configuration, in a better way then we have ATM 
>>>> (expecting them to read the docs). This approach was to include a 
>>>> plugin (that lives in maven central). This plugin would detect 
>>>> missing dependencies that would be available had the correct 
>>>> repository been defined in the settings.xml. There might also be an 
>>>> option to fix the configuration automatically?
>>>
>>> I’m not so sure about this any more.
>>>
>>>>
>>>> Paul.
>>>>
>>>> On 25 Aug 2014, at 13:50, Rafael Benevides <benevides at redhat.com> 
>>>> wrote:
>>>>
>>>>> Hi all,
>>>>>
>>>>> On a look for a continuous improvement on developer user 
>>>>> experience and also because we have been constantly asked to 
>>>>> support issues related to the setup of 
>>>>> https://access.redhat.com/maven-repository to make the quickstarts 
>>>>> work, we want to include the <repository /> definition on 
>>>>> quickstarts pom.xml
>>>>>
>>>>> Actually the Archetypes already have the <repository /> on the 
>>>>> pom.xml file of the generated project.
>>>>>
>>>>> The idea here is to have the <repository /> with the 
>>>>> https://maven.repository.redhat.com/techpreview/all/ defined on 
>>>>> every quickstarts's pom.xml file with a comment on top of it 
>>>>> saying that this approach is not recommended but we included it so 
>>>>> users can test the quickstarts without further setup and that it's 
>>>>> recommended to use settings.xml.
>>>>>
>>>>> It will bring the following advantages:
>>>>> - It will make ease to contributors and users
>>>>> - Simplify the build.
>>>>> - Simplify the Archetype synch process / No need to inject the 
>>>>> repo since it will come from the Quickstarts
>>>>> - We can add a pre-defined comment above the pom.xml repository 
>>>>> definition to explain that we don't recommend that.
>>>>> - We can also add this "comment" to QSTools to check/fix it.
>>>>>
>>>>> As a roadmap for it:
>>>>> - We need to document that at the 
>>>>> https://github.com/jboss-developer/jboss-developer-shared-resources
>>>>> - We need to update the contributing guides
>>>>> - We need to update QSTools to do this update on all quickstarts 
>>>>> for us.
>>>>>
>>>>> Max, Is there any restrictions on JBDS side ?
>>>>>
>>>>> Anyone else have any objections/comments on this $subject ?
>>>>>
>>>>> Thanks
>>>>> -- 
>>>>>
>>>>> Rafael Benevides | Senior Software Engineer
>>>>> JBoss Developer
>>>>> M: +55-61-9269-6576
>>>>>
>>>>> <{a8aabf3a-4467-4e37-9bc5-48b1d7b494a2}_LATAM_RedHat.jpg>
>>>>>
>>>>> Better technology. Faster innovation. Powered by community 
>>>>> collaboration.
>>>>> See how it works at www.redhat.com
>>>>>
>>>>> <linkedin.png> <youtube.png>
>>>>> _______________________________________________
>>>>> jbossdeveloper mailing list
>>>>> jbossdeveloper at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/jbossdeveloper
>>>>
>>>>
>>>> Paul.
>>>>
>>>> -- 
>>>> Paul Robinson
>>>> JBoss Developer Team Lead (www.jboss.org)
>>>>
>>>> JBoss, a Division of Red Hat
>>>> Registered in England and Wales under Company Registration No. 
>>>> 03798903
>>>> Directors:Michael Cunningham (US), Charles Peters (US), Matt Parson 
>>>> (US), Paul Hickey (Ireland)
>>>>
>>>> _______________________________________________
>>>> jbossdeveloper mailing list
>>>> jbossdeveloper at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/jbossdeveloper
>>>
>>
>>
>> Paul.
>>
>> -- 
>> Paul Robinson
>> JBoss Developer Team Lead (www.jboss.org)
>>
>> JBoss, a Division of Red Hat
>> Registered in England and Wales under Company Registration No. 
>> 03798903
>> Directors:Michael Cunningham (US), Charles Peters (US), Matt Parson 
>> (US), Paul Hickey (Ireland)


/max
http://about.me/maxandersen


More information about the jbossdeveloper mailing list