Mailing-List: contact dev-help@directory.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Apache Directory Developers List" <dev@directory.apache.org>
Received-SPF: pass (nike.apache.org: domain of elecharny@gmail.com designates
 74.125.82.44 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:reply-to:user-agent:mime-version:to:subject
         :references:in-reply-to:content-type:content-transfer-encoding;
        b=rN2hSYx5wyyAeOQmmrCWvedsPKa0dNgc95TLrutRY/OHnGGnicbgkRxd0xwwNH8kMz
         8cLBKBm0Ufb/ru6Bd06+Dd6WZHygQFpUvQRU69sq1/bGeKf1e6/RlMowJ7hOhPq0QUoZ
         Wi13qxyxqR93cHY0DY6s4O4A7Rv167r+S8hvc=
Message-ID: <4D836BD6.5070306@gmail.com>
Date: Fri, 18 Mar 2011 15:27:34 +0100
From: Emmanuel Lecharny <elecharny@gmail.com>
Reply-To: elecharny@apache.org
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US;
 rv:1.9.2.13) Gecko/20101207 Thunderbird/3.1.7
MIME-Version: 1.0
To: Apache Directory Developers List <dev@directory.apache.org>
Subject: Re: Pb with mvn eclipse:eclipse and -Dintegration
References: <4D83660D.5000405@gmail.com>
 <40F87FF7-4AB0-4D3C-B747-DEDBFE514B44@marcelot.net>
In-Reply-To: <40F87FF7-4AB0-4D3C-B747-DEDBFE514B44@marcelot.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

On 3/18/11 3:18 PM, Pierre-Arnaud Marcelot wrote:
> Hi Emmanuel,
>
> On 18 mars 2011, at 15:02, Emmanuel Lecharny wrote:
>
>> Hi guys,
>>
>> I'm facing some painful problem with the last modifications made on the build system : as the shared-integ is now depending on the -Dintegration profile to be present to be executed, if you forget to provide this profile when doing a mvn eclipse:eclipse, the .classpath is not generated correctly.
>>
>> So now, to get all the module correctly imported in eclipse, you have to do :
>> mvn eclipse:eclipse -Dintegration
>>
>> Although I understand that it was mandatory to add this profile to get the site generated and the release to be done, I think there are two problems with this approach :
>> - first, no one has been informed about this modification
> This has been discussed in this thread on the ML [1]. You probably didn't see it.

Yeah, my bad. I even remember that you asked me if I was able to run a 
mvn site. I was working for a client the last two weeks, I most 
certainly didn't pay enough attention to those mails.
>> - second, fixing an issue by creating a new one is not a solution.
> I don't think it created an issue, it's just something you need to know before generating Eclipse files.
> Now, I understand it can become a problem when you're used to doing something else for a long time, and having to specify to specify a specific profile can also be counter-intuitive.
It's an issue when you have to provide a profile which is useless when 
generating the eclipse files : you are not splitting the project in two 
areas, one for normal proects, and one for integration projects, when 
injecting all of them in eclipse. It should be done in one pass with no 
argument.

Adding the -Dintegration to mvn eclipse:eclipse is just a workaround, 
and it's bad smell.

>> I'm not blaming anyone here, I just think that when someone is facing such an issue, creating a JIRA is necessary to get everyone informed about the problem, and giving someone an opportunity to track the problem and eventually fix it.
> I think we can rollback on the specific profile creation and go back with old system where everything was generated without the need to add a specific profile.
> We'd then have to try to find another fix for the site generation issue which will block further releases.

yes, probably better. I know it's a pain in the ass, and I hate such 
problems, but frankly, we do mvn eclipse:eclipse way more than site 
generation, and we have to get the frequent operations working as simply 
as possible.

Ok, I created a JIRA so that we can track this problem down.

Thanks a lot guys !

-- 
Regards,
Cordialement,
Emmanuel Lécharny
www.iktek.com