Yes indeed that has already happen:

https://github.com/apache/camel/commit/5f9913390005aa7a672436704bb848c65199e2c8 
<https://github.com/apache/camel/commit/5f9913390005aa7a672436704bb848c65199e2c8>


> On 9 Jan 2021, at 13:46, Andrea Cosentino <anco...@gmail.com> wrote:
> 
> Basically the camel dependencies must be equals to parent pom and it must
> be regenerated starting from catalog. If a dependencies is defined only in
> camel-dependencies it will be override by catalog with the parent pom
> dependedencies
> 
> Il sab 9 gen 2021, 13:44 Andrea Cosentino <anco...@gmail.com> ha scritto:
> 
>> If the impsort version is not defined the full build should fail. I don't
>> have the build, but I suppose it will fail.
>> 
>> Il sab 9 gen 2021, 13:42 Babak Vahdat <babak.vah...@swissonline.ch> ha
>> scritto:
>> 
>>> Hi
>>> 
>>> Where do you see the broken build? I can’t spot it.
>>> 
>>>> On 9 Jan 2021, at 13:31, Andrea Cosentino <anco...@gmail.com> wrote:
>>>> 
>>>> The camel-dependencies are based on parent. And it is autogenerated from
>>>> catalog. So please revert. The build is broken with this commit.
>>>> 
>>>> Il sab 9 gen 2021, 12:42 <bvah...@apache.org> ha scritto:
>>>> 
>>>>> This is an automated email from the ASF dual-hosted git repository.
>>>>> 
>>>>> bvahdat pushed a commit to branch master
>>>>> in repository https://gitbox.apache.org/repos/asf/camel.git
>>>>> 
>>>>> 
>>>>> The following commit(s) were added to refs/heads/master by this push:
>>>>>    new cc34db7  let's declare the impsort-maven-plugin version only in
>>>>> one place
>>>>> cc34db7 is described below
>>>>> 
>>>>> commit cc34db7f3051e37f8aaed34d6964240976955053
>>>>> Author: Babak Vahdat <bvah...@apache.org>
>>>>> AuthorDate: Sat Jan 9 12:41:50 2021 +0100
>>>>> 
>>>>>   let's declare the impsort-maven-plugin version only in one place
>>>>> ---
>>>>> camel-dependencies/pom.xml | 2 +-
>>>>> parent/pom.xml             | 1 -
>>>>> pom.xml                    | 2 +-
>>>>> 3 files changed, 2 insertions(+), 3 deletions(-)
>>>>> 
>>>>> diff --git a/camel-dependencies/pom.xml b/camel-dependencies/pom.xml
>>>>> index 5854a67..6f6d7c7 100644
>>>>> --- a/camel-dependencies/pom.xml
>>>>> +++ b/camel-dependencies/pom.xml
>>>>> @@ -270,7 +270,7 @@
>>>>>    <ical4j-version>1.0.7</ical4j-version>
>>>>>    <icu4j-version>65.1</icu4j-version>
>>>>>    <ignite-version>2.9.1</ignite-version>
>>>>> -    <impsort-maven-plugin-version>1.3.2</impsort-maven-plugin-version>
>>>>> +    <impsort-maven-plugin-version>1.4.1</impsort-maven-plugin-version>
>>>>>    <infinispan-version>11.0.8.Final</infinispan-version>
>>>>>    <influx-guava-version>20.0</influx-guava-version>
>>>>>    <influx-java-driver-version>2.20</influx-java-driver-version>
>>>>> diff --git a/parent/pom.xml b/parent/pom.xml
>>>>> index 056337b..99eb2cb 100644
>>>>> --- a/parent/pom.xml
>>>>> +++ b/parent/pom.xml
>>>>> @@ -250,7 +250,6 @@
>>>>>        <ical4j-version>1.0.7</ical4j-version>
>>>>>        <icu4j-version>65.1</icu4j-version>
>>>>>        <ignite-version>2.9.1</ignite-version>
>>>>> -
>>> <impsort-maven-plugin-version>1.3.2</impsort-maven-plugin-version>
>>>>>        <infinispan-version>11.0.8.Final</infinispan-version>
>>>>>        <influx-java-driver-version>2.20</influx-java-driver-version>
>>>>>        <influx-guava-version>20.0</influx-guava-version>
>>>>> diff --git a/pom.xml b/pom.xml
>>>>> index dc7825d..9f14176 100644
>>>>> --- a/pom.xml
>>>>> +++ b/pom.xml
>>>>> @@ -117,7 +117,7 @@
>>>>>        <!-- we need to override the version inherited from Apache POM
>>>>> for modules that use this POM as parent -->
>>>>> 
>>>>> <surefire.version>${maven-surefire-plugin-version}</surefire.version>
>>>>> 
>>>>> <formatter-maven-plugin.version>2.12.1</formatter-maven-plugin.version>
>>>>> -
>>> <impsort-maven-plugin.version>1.4.1</impsort-maven-plugin.version>
>>>>> +
>>> <impsort-maven-plugin-version>1.4.1</impsort-maven-plugin-version>
>>>>>        <java-diff-utils-version>4.7</java-diff-utils-version>
>>>>> 
>>> <maven-bundle-plugin-version>4.2.1</maven-bundle-plugin-version>
>>>>> 
>>>>> 
>>>>> 
>>> 
>>> 

Reply via email to