Mailing-List: contact axis-c-user-help@ws.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Apache AXIS C User List" <axis-c-user@ws.apache.org>
Message-ID: <48410.220.247.241.95.1081257780.squirrel@webmail3.pair.com>
In-Reply-To: <20040406035124.49738.qmail@web40606.mail.yahoo.com>
References: <36796.220.247.245.240.1080204078.squirrel@webmail4.pair.com>
    <20040406035124.49738.qmail@web40606.mail.yahoo.com>
Date: Tue, 6 Apr 2004 09:23:00 -0400 (EDT)
Subject: Re: Some comments on source distibution for R1.1
From: damitha@opensource.lk
To: "Apache AXIS C User List" <axis-c-user@ws.apache.org>,
  axis-c-dev@ws.apache.org
User-Agent: SquirrelMail/1.4.2
MIME-Version: 1.0
Content-Type: text/plain;charset=iso-8859-1
Content-Transfer-Encoding: 8bit
Importance: Normal

Hi,

Following I've given some of my ideas of your attached file.

1. Please add to configure.ac the following to help Xerces parser users
#CPPFLAGS="-Wall -Wshadow -DUSE_XERCES_PARSER"
#LDFLAGS="-s -L$AXISCPP_HOME/lib/xercesc -lxerces-c -lstdc++"
---------------------
This is done. I've mentioned some of the problems related to LDFLAGS in a
seperate mail.


1. Update the INSTALL file in the root source distribution folder to
reflect the
latest simple set of install steps
---------------------
This need to be done

2. when ./autogen.sh is run
Makefile.am: required file `./NEWS' not found
Makefile.am: required file `./README' not found
Makefile.am: required file `./AUTHORS' not found
Makefile.am: required file `./ChangeLog' not found
Add (empty) files for them to eliminate warnings
Also meaningful content could be added to them as well
----------------------
+1

3. runconfig script does not have runnig permissions. chmod 755 runconfig
(for
the benefit of those who use ./runconfig)
build.sh does not have running permissions (for those who wish to use
./build.sh)
------------------------------------------
+1

4. There are many compiler warnings. Can we eliminate them.
--------------------------------------
+1

5. It is critical to mention that most of the build problems are related
to not
seeting AXISCPP_HOME. (we can set this in buld.sh if it is not set already)
------------------------------------------
I think it is not an over burden to set some environment varibles when
installing an application. For example xercesc, tomcat and several favmous
application use this approach

6. How do I instruct configure to use either Expat or Xerces?


7. Installation guide is missing in docs


8. Can we get rid of linux, windows directories in docs directory and include
platform options within a single document, with seperate sections
--------------------
I don't understand why this is important. However xercesc also have all in
single doc with links.

9. Lets make the default build to be Apache 2 in configure.ac (as most of the
OSes bundle apache 2)
------------------
this is done.

10. We need to have a #if you use apache1.3 comment in configure.ac

--------------
done

11. Add LDFLAGS and CPPFLAGS in configure.ac so that swithcing parsers
becomes simple.
------------------
done

12. Can we add the parser selection as a parameter to configure script? When
./configure --help is run, it does not show anything related to the parser
selection. I think it should.
----------------------
this is good. but I don't know how to do this

13. We have to document the Expat specific steps and Xerces related steps
sperately. (this is a critical requirement of parser abstraction layer)
-------------------------
this should be done. +1

14.Remove the 'xerces' folder in include (this confuces with xercesc)
------------------------
removed

15. In deploy/axiscpp.conf I had to change apache to apache2 (apache2 is the
default install folder) Either we have to document this change or have a
mechanism to autogenerate during configuration
----------------------------
We need to document this.

16. What about users that use apache that comes bundled with OS installation?
Its good to have documentation for them as well
---------------------------
I've tested in such an environment. Steps are intuitive.

17. deploy/conf/server.wsdd reffers to /usr/local/apache2 (unlike
deploy/axiscpp.conf). Have to consider documenting for users with bundled
apache
and apache 1.3

18. Can we make samples and simple axis sever tobe bulit during intial
builsstep?
Then we can eliminate many redundent steps.
-------------------
I thought that building samples seperately is better. But I've seen expat
do as you said.

19. In configure.ac include the settings for Xerces parser as well and
keep them
commented (because expat is the default)
----------------------
commented???

20. In include directory Xerces subfolder is xercesc and in lib directory
Xerces
subfolder is xerces-c.
This is inconsistant. Lets use xercesc for both as it is the one used by
Xerces itself.
(But the lib name is xerces-c)
----------------------
done


> Hi All,
>     I was trying out the source from the CVS over the weekend and came up
> with few suggestions to
> make the source distibution more user friendly.
>
>     Most of the suggestions are focused for the Linux platform.
>
>     Please see the attached text file.
> Thanks,
> Samisa...
>
> __________________________________
> Do you Yahoo!?
> Yahoo! Small Business $15K Web Design Giveaway
> http://promotions.yahoo.com/design_giveaway/