Le 11/12/14 19:36, Stefan Seelmann a écrit :
> On 12/11/2014 05:59 PM, Emmanuel Lécharny wrote:
>> Hi !
>>
>> today, I spent some time on checking *all* the dependencies for LDAP
>> API, and all the imported and exported packages. I removed a few useless
>> dependencies.
>>
>> One whic I think might be troublesome is a dependency on xml-apis 2.0.2
>> in dsml/parser, when it internally uses dom4j, which depends on xml-apis
>> 1.0.b2. That *might* explain the pb we have when trying to build Studio
>> with the latest API version (not sure though).
>>
>> I still have a few modules to check, then I'll have to test the build
>> with a API 1.0.0-M27-SNAPSHOT, which is everything but simple (I'm
>> afraid I'lm have to update the various manifest with the generated
>> versions, which is all but funny...)
>>
>> Stefan, do you have a better idea ?
> sed for the rescue?
>
> find . -name MANIFEST.MF | xargs sed -i
> 's/\(org\.apache\.directory\.api.*\)\(1\.0\.0\.M24\)/\11.0.0.M27-SNAPSHOT/g'
>
>
> The p2-maven-plugin we use for generating the P2 repo of the
> eclipse-target-platform doesn't support "-SNAPSHOT" versions but
> replaces "-SNAPSHOT" with the current timestamp. 

I will try that.

> There is an issue [1]
> to change that. As a quick fix I cloned the source, did the following
> change, built it and use that patched version in
> eclipse-target-platform/pom.xml:
>
> diff --git a/src/main/java/org/reficio/p2/utils/JarUtils.java
> b/src/main/java/org/reficio/p2/utils/JarUtils.java
> index 67be51f..33889a6 100644
> --- a/src/main/java/org/reficio/p2/utils/JarUtils.java
> +++ b/src/main/java/org/reficio/p2/utils/JarUtils.java
> @@ -137,7 +137,7 @@ public class JarUtils {
>          } else if (version.contains(OSGI_SNAPSHOT_POSTFIX)) {
>              tweakedVersion =
> tweakedVersion.replace(OSGI_SNAPSHOT_POSTFIX, "." + getTimeStamp());
>          }
> -        return tweakedVersion;
> +        return version;
>      }

Sounds like a workaround. But at some point, we should get rid of the
hand-drafter manifests, and make them generated by the felix plugin.


Reply via email to