Embedding chemistry-opencmis-server-support.jar in 
chemistry-opencmis-server-bindings.war and remove 
chemistry-opencmis-dist-server.zip
---------------------------------------------------------------------------------------------------------------------------------------

                 Key: CMIS-335
                 URL: https://issues.apache.org/jira/browse/CMIS-335
             Project: Chemistry
          Issue Type: Improvement
          Components: build&release
    Affects Versions: 0.2.0-incubating
            Reporter: Gabriele Columbro
            Assignee: Gabriele Columbro
            Priority: Minor
             Fix For: OpenCMIS 0.3.0


Would be good if the server-bindings.war could be a working webapp and for most 
usages this will require inclusion of the server-support.jar (even not being a 
strict "runtime" dependency. 

The simplest option here is:
- Add the JAR as a dependency for the already released WAR
- Make the WAR available in the /dist commodity packages
- deprecate the -server.zip custom assembly 

See below details of the ML discussion:

~~~o0o~~~


Sorry to have not really been clear on this topic.

The proposed target structure of opencmis-dist-0.3.0-server.zip I have in mind 
is like this:
LICENSE          
DEPENDENCIES
NOTICE           
chemistry-opencmis-server-bindings-0.3.0-SNAPSHOT.war  

with chemistry-opencmis-server-bindings-0.3.0-SNAPSHOT.war:

META-INF
   LICENSE          
   DEPENDENCIES
   NOTICE              
WEB-INF
   classes
      repository.properties
   lib
       activation-1.1.jar
       chemistry-opencmis-commons-api-0.2.0-incubating.jar
       chemistry-opencmis-commons-impl-0.2.0-incubating.jar
       chemistry-opencmis-server-bindings-0.2.0-incubating.jar
       chemistry-opencmis-server-support-0.3.0-SNAPSHOT.jar     (!!! new !!!)
       commons-codec-1.4.jar
       commons-logging-1.1.1.jar
       dir.txt
       jaxb-api-2.1.jar
       jaxb-impl-2.1.11.jar
       jaxws-api-2.1.jar
       jaxws-rt-2.1.7.jar
       mimepull-1.3.jar
       resolver-20050927.jar
       saaj-api-1.3.jar
       saaj-impl-1.3.3.jar
       stax-api-1.0.1.jar
       stax-api-1.0.jar
       stax-ex-1.2.jar
       streambuffer-0.9.jar
       wstx-asl-3.2.3.jar

   web.xml
   sun-jaxws.xml
   wsdl
        CMIS-Core.xsd
        CMIS-Messaging.xsd    
        CMISWS-Service.wsdl
        xml.xsd               


If we want to directly make this .war shippable this would be perfectly fine 
with me. My point is
to have something to download if you want build a server without maven that 
contains all necessary
dependencies. It is just the counterpart to 
chemistry-opencmis-client-impl-0.2.0-incubating-with-dependencies.zip for the 
server side.

Would this work for you?

BTW is the directory META-INF/maven intended in this war?

Jens




-----Original Message-----
From: Gabriele Columbro [mailto:[email protected]] 
Sent: Dienstag, 22. März 2011 17:24
To: [email protected]
Subject: Re: Going for OpenCMIS release 0.3.0 today

Hey Jens,
was going to implement this change for the release, but I still need to grasp 
the overall meaning of it. 

ATM the -opencmis-dist-0.3.0-server.zip contains:

$ unzip chemistry-opencmis-dist-0.3.0-SNAPSHOT-server.zip
Archive:  chemistry-opencmis-dist-0.3.0-SNAPSHOT-server.zip
 inflating: LICENSE          
 inflating: DEPENDENCIES     
 inflating: NOTICE           
 inflating: chemistry-opencmis-server-bindings-0.3.0-SNAPSHOT.war  
 inflating: chemistry-opencmis-server-support-0.3.0-SNAPSHOT.jar 

IIUC you would like the build create ZIP a -opencmis-dist-0.3.0-server.zip as 
such:

- LICENSE
- DEPENDENCIES
- NOTICE
- chemistry-opencmis-server-bindings-0.3.0-SNAPSHOT.war  (with WEB-INF/lib of 
this WAR containing also chemistry-opencmis-server-support-0.3.0-SNAPSHOT.jar)

Is this correct? Or (like it seems in the last email) you want a WEB-INF/lib in 
side the ZIP itself?

In general, I don't really see the point of creating a ZIP which only contains 
a WAR + LICENSE/DEPENDENCIES files inside (BTW also contained in the WARs 
META-INF).
Can't we just add server-support.jar as a dependency for server-bindings.war 
(so it will be included in WEB-INF/lib) and maybe make the WAR available on 
/dist, so you can simply download and run it? (in this case -server.zip will 
simply be dropped)

Would this match your requirement?

Thanks,
Gab



On Mar 21, 2011, at 4:23 PM, Jens Hübel wrote:

What I meant was: 
- keep the -opencmis-dist-0.3.0-server.zip
- but within that package move the file 
chemistry-opencmis-server-support-0.3.0.jar 
 from top-level to directory WEB-IN/lib within that zip

A very minor change just to make the zip following a web application structure 
(could rename this
to .war as well, but .zip and .tar.gz are more standard for downloads). The 
idea is that you get a
running web application after unzipping.

Jens


-----Original Message-----
From: Gabriele Columbro [mailto:[email protected]] 
Sent: Montag, 21. März 2011 15:32
To: [email protected]
Subject: Re: Going for OpenCMIS release 0.3.0 today

Hey Jens,
sorry for not having come back to you on this one earlier.

If I understand correctly we could just remove the "-server.zip"
package and simply rely on the fact that -server-support.jar" is
shipped in the server-bindings.war . Correct?

I'll do this change prior to releasing, if there's no further objection.

Thanks,
Gab


On Mon, Mar 21, 2011 at 2:43 PM, Jens Hübel <[email protected]> wrote:
No objections from my side, but did not get any feedback to this one here 
(would be great to get this included)

Jens

Subject: Minor change request in our packaging for next release

Hi Gab,

could you do a small change in the package 
chemistry-opencmis-dist-0.2.0-incubating-server.zip?

The zip file currently contains 
chemistry-opencmis-server-bindings-0.2.0-incubating.war and 
chemistry-opencmis-server-support-0.2.0-incubating.jar.

The .jar file is not much different to the other jars that we release in the 
war file in WEB-INF/lib. It would make more sense and be more consistent if we 
just ship this together with the others in WEB-INF/lib of the .war file. This 
would be less confusing for people who want to build a server. We could in 
future improve documentation and indicate which jars are needed and which are 
optional. But 99% probably don't care about the few bytes needed on disk.

I discussed this already briefly with Florian and he agreed to my suggestion. 
Could you perform this small change for the upcoming 0.3 release?

Jens

-----Original Message-----
From: Gabriele Columbro [mailto:[email protected]]
Sent: Montag, 21. März 2011 14:33
To: [email protected]
Subject: Going for OpenCMIS release 0.3.0 today

Hi guys,
just a heads up since I finally found some time to shoot at the first
TLP release for OpenCMIS.

I'll probably go for it in in 2 hours from now, so, if you have
anything to be checked in prior to that, I urge you to go for it if
you are confident enough.

Any issue which would block the release according to you?

Thanks,
Gab

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to