Richard S. Hall  wrote
> Is the info in the DEPS file for maven-scr-plugin correct? It lists
> stuff that I don't see in the deps section of the pom file.

Yes, I think they are correct - these are transitive dependencies from
the generator, except slf4j.

> 
> SCR Ant and SCR Generate also list slf4j in their DEPS file, but I don't
> see it in their pom files. Is that accurate?
No, I don't think so - but I don't think that this is problem or blocker.

I just removed the slf4j from all deps files for the future.


> It looks like the change log wasn't updated for SCR Generator.
It's recent except I forgot to change the version from 1.0.2 to 1.1.0.
So it contains the correct changes. (Fixed in svn now)

> General comment, since all of the DEPS files are hand generated, if you
> use the suggested template
> (http://felix.apache.org/site/dependencies-file-template.html) you could
> simplify them a little, since the template focuses on third-party
> dependencies, which eliminates the repetition of "included" and "used"
> Apache-developed software.

Yes, right - actually I planned to use the new mechanism introduced by
Felix (FELIX-1747) but that would have required a release of the parent
pom first.

Thanks
Carsten

> 
> -> richard
> 
> On 3/9/11 9:17, Carsten Ziegeler wrote:
>> Hi,
>>
>> this vote is about our SCR Tools
>>
>> - Apache Felix SCR Maven Plugin 1.7.0
>> - Apache Felix SCR Annotations 1.5.0
>> - Apache Felix SCR Ant Task 1.1.0
>> - Apache Felix SCR Generator 1.1.0
>>
>>
>> We solved several important issues in this release:
>> https://issues.apache.org/jira/browse/FELIX/fixforversion/12315492
>>
>>
>> Staging repository:
>> https://repository.apache.org/content/repositories/orgapachefelix-004/
>>
>> You can use this UNIX script to download the release and verify the
>> signatures:
>> http://svn.apache.org/repos/asf/felix/trunk/check_staged_release.sh
>>
>> Usage:
>> sh check_staged_release.sh 004 /tmp/felix-staging
>>
>> Please vote to approve this release:
>>
>> [ ] +1 Approve the release
>> [ ] -1 Veto the release (please provide specific comments)
>>
>> This vote will be open for 72 hours.
>>
>> Regards
>> Carsten
> 


-- 
Carsten Ziegeler
[email protected]

Reply via email to