Re: [POLL] Bug 48804

2010-04-26 Thread Antoine Levy Lambert
Stefan Bodewig wrote: On 2010-04-23, Martijn Kruithof wrote: I agree, however it may be good to update the 1.8.0 manual with a warning that the behavior is currently not as intended and will change, published as soon as the solution is changed. The manual is a tagged svn revisio. Gi

Re: [POLL] Bug 48804

2010-04-26 Thread Antoine Levy Lambert
Dominique Devienne wrote: On Fri, Apr 23, 2010 at 4:29 AM, Stefan Bodewig wrote: [...] You can't extend an imported extension point with a target from the importing build file (which is the primary use-case, really). Right. [...] Personally I think the changed behavior isn't a ba

Re: [POLL] Bug 48804

2010-04-26 Thread Peter Reilly
+1 for fixing Peter On Fri, Apr 23, 2010 at 3:37 PM, Stefan Bodewig wrote: > On 2010-04-23, Dominique Devienne wrote: > >> On Fri, Apr 23, 2010 at 4:29 AM, Stefan Bodewig wrote: > >>> [...] Personally I think the changed behavior isn't a bad thing, the old >>> behavior isn't documented and we s

Re: [POLL] Bug 48804

2010-04-24 Thread Stefan Bodewig
On 2010-04-23, Martijn Kruithof wrote: > I agree, however it may be good to update the 1.8.0 manual with a > warning that the behavior is currently not as intended and will > change, published as soon as the solution is changed. The manual is a tagged svn revisio. Given that Antoine intends to

Re: [POLL] Bug 48804

2010-04-23 Thread Martijn Kruithof
I agree, however it may be good to update the 1.8.0 manual with a warning that the behavior is currently not as intended and will change, published as soon as the solution is changed. br martijn On 23/04/2010 2:29 AM, Stefan Bodewig wrote: Hi all, currently extension-point and import don't

Re: [POLL] Bug 48804

2010-04-23 Thread Bruce Atherton
I think that since the code doesn't address the primary usecase, that trumps pretty-well all considerations. It is hard to imagine how someone would be relying on the order of the extension point/extension evaluation in order to avoid the extension point. Since the behaviour isn't documented a

Re: [POLL] Bug 48804

2010-04-23 Thread Matt Benson
On Apr 23, 2010, at 4:29 AM, Stefan Bodewig wrote: Hi all, currently extension-point and import don't play together like they are supposed to. You can't extend an imported extension point with a target from the importing build file (which is the primary use-case, really). Attached to this

Re: [POLL] Bug 48804

2010-04-23 Thread Stefan Bodewig
On 2010-04-23, Dominique Devienne wrote: > On Fri, Apr 23, 2010 at 4:29 AM, Stefan Bodewig wrote: >> [...] Personally I think the changed behavior isn't a bad thing, the old >> behavior isn't documented and we shouldn't even try to keep it. > I agree. Fixing the primary use-case trumps this pa

Re: [POLL] Bug 48804

2010-04-23 Thread Dominique Devienne
On Fri, Apr 23, 2010 at 4:29 AM, Stefan Bodewig wrote: > [...] You can't extend an imported extension point with a target > from the importing build file (which is the primary use-case, really). Right. > [...] Personally I think the changed behavior isn't a bad thing, the old > behavior isn't do

[POLL] Bug 48804

2010-04-23 Thread Stefan Bodewig
Hi all, currently extension-point and import don't play together like they are supposed to. You can't extend an imported extension point with a target from the importing build file (which is the primary use-case, really). Attached to this bug is a patch that fixes the problem (including an AntUn