Return-Path: Delivered-To: apmail-ws-axis-c-dev-archive@www.apache.org Received: (qmail 78791 invoked from network); 19 Feb 2007 19:07:27 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 19 Feb 2007 19:07:27 -0000 Received: (qmail 40750 invoked by uid 500); 19 Feb 2007 19:07:34 -0000 Delivered-To: apmail-ws-axis-c-dev-archive@ws.apache.org Received: (qmail 40734 invoked by uid 500); 19 Feb 2007 19:07:34 -0000 Mailing-List: contact axis-c-dev-help@ws.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: List-Id: "Apache AXIS C Developers List" Reply-To: "Apache AXIS C Developers List" Delivered-To: mailing list axis-c-dev@ws.apache.org Received: (qmail 40722 invoked by uid 99); 19 Feb 2007 19:07:34 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 19 Feb 2007 11:07:34 -0800 X-ASF-Spam-Status: No, hits=2.0 required=10.0 tests=HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (herse.apache.org: domain of jaredhanson@gmail.com designates 66.249.90.179 as permitted sender) Received: from [66.249.90.179] (HELO ik-out-1112.google.com) (66.249.90.179) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 19 Feb 2007 11:07:23 -0800 Received: by ik-out-1112.google.com with SMTP id c30so142423ika for ; Mon, 19 Feb 2007 11:07:01 -0800 (PST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; b=qmAafLiI+b/J7ZTXBEZv6EMBbCUqn0t3lxBU2fZsiRCCSCSIe3onFVZYhL838oId/Lxeq2HcztHQgHGjDeLYvWsaDBPp276aKUmYuuFsIMPyi2WSTkpGTRVhcRDXWQISpTmq5Im+Gcq8+22v8N78ya9MwgpiBnNZaJdA9V3sEVk= Received: by 10.114.174.2 with SMTP id w2mr3028832wae.1171912020385; Mon, 19 Feb 2007 11:07:00 -0800 (PST) Received: by 10.115.46.7 with HTTP; Mon, 19 Feb 2007 11:07:00 -0800 (PST) Message-ID: <8c807c880702191107r1d92f3d8kdf2d6573fae53bc4@mail.gmail.com> Date: Mon, 19 Feb 2007 11:07:00 -0800 From: "Jared Hanson" To: "Apache AXIS C Developers List" Subject: Re: Code for Axis2/C Static Deployment Engine In-Reply-To: <87tzxkahzg.fsf@gmail.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_42618_11579598.1171912020211" References: <8c807c880702161536r4fc8da85o1d72a41636b60076@mail.gmail.com> <87tzxkahzg.fsf@gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_42618_11579598.1171912020211 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline Glad to hear you're interested in the code. It's one of my top priorities to get this building against svn head this week. I'll be submitting patches as I complete tasks. One question I did have: Right now I split out the static deployment engine code, so it is completely outside of the axis2c code base. However, it turns out that about 95% of the code is a copy of what is in axis2c (configuration parsing, etc). As I work off head, I'd like to avoid this duplication. One option that is available is just using a #ifdef directly in the code, usually in the deployment module. For example: #ifdef AXIS2_STATIC_DEPLOY /* Do static deployment step */ #else /* Do normal dynamic deployment step */ #endif Another option is to make a separate static deployment module. This option avoids the #ifdef pragmas, but would require more refactoring in order to move the common code to a place that can be shared by both the static and dynamic deployment modules. If anyone has any opinions on this, please make them known. Also, as you get a chance to review the code I uploaded, let me know any changes you would like made. - Jared On 2/17/07, Dinesh Premalal wrote: > > Hi Jared, > > "Jared Hanson" writes: > > I've uploaded the code for the Axis2/C static deployment engine that > > I've been referring to. It can be downloaded from: > > http://www.asperasoft.com/developer/axis2c/axis2-static-deploy.tar.gz > Downloaded sources, Give us some time look through it. > > What is in the package was built against 0.95. As mentioned, there are > some > > issues when building against the trunk. I intend to fix those. > As your previous mail mentioned, there is some refactoring going on the > code base. We removed some of the ops because there was some > performance hiccups. > > Lets try to get it build against the trunk, and put it in the code > base. After that you will be able to do your further development > easily. If we could include, static deployment engine for Axis2/C 1.0 > release that would be great. > > If there is any interest in incorporating this into the project, > > that would be welcome. > Definitely !, we are open to embrace any constructive ideas or > contributions. > >Please ask if you have any questions. Any suggestions are > > appreciated. > Appreciate your effort on Static Deployment Engine. Lets try to get it > on svn head soon. > > thanks, > Dinesh > -- > Dinesh Premalal > dinesh@wso2.com > WSO2, Inc.; http://www.wso2.com/ > GPG Key ID : A255955C > GPG Key Finger Print : C481 E5D4 C27E DC34 9257 0229 4F44 266E A255 955C > > --------------------------------------------------------------------- > To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org > For additional commands, e-mail: axis-c-dev-help@ws.apache.org > > ------=_Part_42618_11579598.1171912020211 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline Glad to hear you're interested in the code.

It's one of my top priorities to get this building against svn head this week.  I'll be submitting patches as I complete tasks.

One question I did have:

Right now I split out the static deployment engine code, so it is completely outside of the axis2c code base.  However, it turns out that about 95% of the code is a copy of what is in axis2c (configuration parsing, etc).

As I work off head, I'd like to avoid this duplication.  One option that is available is just using a #ifdef directly in the code, usually in the deployment module.

For example:

#ifdef AXIS2_STATIC_DEPLOY
/* Do static deployment step */
#else
/* Do normal dynamic deployment step */
#endif

Another option is to make a separate static deployment module.  This option avoids the #ifdef pragmas, but would require more refactoring in order to move the common code to a place that can be shared by both the static and dynamic deployment modules.

If anyone has any opinions on this, please make them known.

Also, as you get a chance to review the code I uploaded, let me know any changes you would like made.

- Jared


On 2/17/07, Dinesh Premalal <xydinesh@gmail.com> wrote:
Hi Jared,

"Jared Hanson" <jaredhanson@gmail.com> writes:
> I've uploaded the code for the Axis2/C static deployment engine that
> I've been referring to.  It can be downloaded from:
> http://www.asperasoft.com/developer/axis2c/axis2-static-deploy.tar.gz
Downloaded sources, Give us some time look through it.
> What is in the package was built against 0.95.  As mentioned, there are some
> issues when building against the trunk.  I intend to fix those.
As your previous mail mentioned, there is some refactoring going on the
code base. We removed some of the ops because there was some
performance hiccups.

Lets try to get it build against the trunk, and put it in the code
base. After that you will be able to do your further development
easily. If we could include, static deployment engine for Axis2/C 1.0
release that would be great.
> If there is any interest in incorporating this into the project,
> that would be welcome.
Definitely !, we are open to embrace any constructive ideas or
contributions.
>Please ask if you have any questions.  Any suggestions are
> appreciated.
Appreciate your effort on Static Deployment Engine. Lets try to get it
on svn head soon.

thanks,
Dinesh
--
Dinesh Premalal
dinesh@wso2.com
WSO2, Inc.; http://www.wso2.com/
GPG Key ID : A255955C
GPG Key Finger Print : C481 E5D4 C27E DC34 9257  0229 4F44 266E A255 955C

---------------------------------------------------------------------
To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-c-dev-help@ws.apache.org


------=_Part_42618_11579598.1171912020211--