You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by Oliver Rossmueller <ol...@tuxerra.com> on 2004/11/01 23:43:38 UTC

Re: cvs commit: incubator-myfaces/build build.xml

Sylvain,

the api classes are in myfaces-jsf-api.jar now, the split was done on 
occasion in July (see revision 1.48/1.49 of build.xml, search for 
subject '[Myfaces-develop] JAR-Files' in the 
myfaces-develop@lists.sourceforge.net archive)  so please revert this 
modification.

Oliver


svieujot@apache.org wrote:

>svieujot    2004/11/01 08:21:46
>
>  Modified:    build    build.xml
>  Log:
>  Add the api.classes to myfaces.jar
>  
>  Revision  Changes    Path
>  1.58      +2 -2      incubator-myfaces/build/build.xml
>  
>  Index: build.xml
>  ===================================================================
>  RCS file: /home/cvs/incubator-myfaces/build/build.xml,v
>  retrieving revision 1.57
>  retrieving revision 1.58
>  diff -u -r1.57 -r1.58
>  --- build.xml	26 Oct 2004 12:41:36 -0000	1.57
>  +++ build.xml	1 Nov 2004 16:21:46 -0000	1.58
>  @@ -119,9 +119,9 @@
>               </manifest>
>   
>               <!-- classes -->
>  -            <!--fileset dir="${api.classes}"
>  +            <fileset dir="${api.classes}"
>                        includes="**"
>  -                     excludes=".dependency-info/**/*"/-->
>  +                     excludes=".dependency-info/**/*"/>
>               <fileset dir="${share.classes}"
>                        includes="**"
>                        excludes=".dependency-info/**/*"/>
>  
>  
>  
>  
>


Re: cvs commit: incubator-myfaces/build build.xml

Posted by Sylvain Vieujot <sv...@apache.org>.
Ok, I made the changes.

It works for me, but please test it and report me any problem.

Sylvain.

On Tue, 2004-11-02 at 05:54, Oliver Rossmueller wrote:

> Sylvain Vieujot wrote:
> 
> > Done.
> > Sorry for that.
> >
> > Maybe we should anyway make a package that has everything, like the 
> > javamail distribution that has several jar files (imap.jar, pop.jar, 
> > ...) and one big mail.jar that has everything and is way easier to 
> > install.
> >
> > With our file names, a simple guy like me thinks that :
> > myfaces-jsf-api.jar contains the api classes.
> > myfaces-components.jar contains the myfaces extensions.
> > myfaces.jar contains everything.
> >
> > So, why not have this scheme :
> > myfaces-jsf-api.jar contains the api classes (no change)
> > myfaces-impl.jar contains the standard jsf implementation, without the 
> > extensions (same as jsf-impl.jar from the RI)
> > myfaces-extensions.jar contains the myfaces extensions
> > and a myfaces.jar that contains everything.
> >
> +1
> 
> Oliver
> 
> > I think this would have the benefits of a clear modular split, with 
> > file names similar to the RI ones, and an easy to use package for 
> > those that don't want to mix the RI with myfaces extensions.
> >
> > By the way, right now, it seems that myfaces-components.jar has the 
> > components ressources, but that myfaces.jar doesn't has them.
> > Is this intended ?
> >
> > Sylvain.
> >
> > On Mon, 2004-11-01 at 18:43, Oliver Rossmueller wrote:
> >
> >>/Sylvain,
> >>
> >>the api classes are in myfaces-jsf-api.jar now, the split was done on 
> >>occasion in July (see revision 1.48/1.49 of build.xml, search for 
> >>subject '[Myfaces-develop] JAR-Files' in the 
> >>myfaces-develop@lists.sourceforge.net archive)  so please revert this 
> >>modification.
> >>
> >>Oliver
> >>
> >>
> >>svieujot@apache.org wrote:
> >>
> >>>svieujot    2004/11/01 08:21:46
> >>>
> >>>  Modified:    build    build.xml
> >>>  Log:
> >>>  Add the api.classes to myfaces.jar
> >>>  
> >>>  Revision  Changes    Path
> >>>  1.58      +2 -2      incubator-myfaces/build/build.xml
> >>>  
> >>>  Index: build.xml
> >>>  ===================================================================
> >>>  RCS file: /home/cvs/incubator-myfaces/build/build.xml,v
> >>>  retrieving revision 1.57
> >>>  retrieving revision 1.58
> >>>  diff -u -r1.57 -r1.58
> >>>  --- build.xml	26 Oct 2004 12:41:36 -0000	1.57
> >>>  +++ build.xml	1 Nov 2004 16:21:46 -0000	1.58
> >>>  @@ -119,9 +119,9 @@
> >>>               </manifest>
> >>>   
> >>>               <!-- classes -->
> >>>  -            <!--fileset dir="${api.classes}"
> >>>  +            <fileset dir="${api.classes}"
> >>>                        includes="**"
> >>>  -                     excludes=".dependency-info/**/*"/-->
> >>>  +                     excludes=".dependency-info/**/*"/>
> >>>               <fileset dir="${share.classes}"
> >>>                        includes="**"
> >>>                        excludes=".dependency-info/**/*"/>
> >>>  
> >>>  
> >>>  
> >>>  
> >>>/
> >>
> > Sylvain.
> >
> >
> >  
> >


Sylvain.


Re: cvs commit: incubator-myfaces/build build.xml

Posted by Oliver Rossmueller <ol...@tuxerra.com>.
Sylvain Vieujot wrote:

> Done.
> Sorry for that.
>
> Maybe we should anyway make a package that has everything, like the 
> javamail distribution that has several jar files (imap.jar, pop.jar, 
> ...) and one big mail.jar that has everything and is way easier to 
> install.
>
> With our file names, a simple guy like me thinks that :
> myfaces-jsf-api.jar contains the api classes.
> myfaces-components.jar contains the myfaces extensions.
> myfaces.jar contains everything.
>
> So, why not have this scheme :
> myfaces-jsf-api.jar contains the api classes (no change)
> myfaces-impl.jar contains the standard jsf implementation, without the 
> extensions (same as jsf-impl.jar from the RI)
> myfaces-extensions.jar contains the myfaces extensions
> and a myfaces.jar that contains everything.
>
+1

Oliver

> I think this would have the benefits of a clear modular split, with 
> file names similar to the RI ones, and an easy to use package for 
> those that don't want to mix the RI with myfaces extensions.
>
> By the way, right now, it seems that myfaces-components.jar has the 
> components ressources, but that myfaces.jar doesn't has them.
> Is this intended ?
>
> Sylvain.
>
> On Mon, 2004-11-01 at 18:43, Oliver Rossmueller wrote:
>
>>/Sylvain,
>>
>>the api classes are in myfaces-jsf-api.jar now, the split was done on 
>>occasion in July (see revision 1.48/1.49 of build.xml, search for 
>>subject '[Myfaces-develop] JAR-Files' in the 
>>myfaces-develop@lists.sourceforge.net archive)  so please revert this 
>>modification.
>>
>>Oliver
>>
>>
>>svieujot@apache.org wrote:
>>
>>>svieujot    2004/11/01 08:21:46
>>>
>>>  Modified:    build    build.xml
>>>  Log:
>>>  Add the api.classes to myfaces.jar
>>>  
>>>  Revision  Changes    Path
>>>  1.58      +2 -2      incubator-myfaces/build/build.xml
>>>  
>>>  Index: build.xml
>>>  ===================================================================
>>>  RCS file: /home/cvs/incubator-myfaces/build/build.xml,v
>>>  retrieving revision 1.57
>>>  retrieving revision 1.58
>>>  diff -u -r1.57 -r1.58
>>>  --- build.xml	26 Oct 2004 12:41:36 -0000	1.57
>>>  +++ build.xml	1 Nov 2004 16:21:46 -0000	1.58
>>>  @@ -119,9 +119,9 @@
>>>               </manifest>
>>>   
>>>               <!-- classes -->
>>>  -            <!--fileset dir="${api.classes}"
>>>  +            <fileset dir="${api.classes}"
>>>                        includes="**"
>>>  -                     excludes=".dependency-info/**/*"/-->
>>>  +                     excludes=".dependency-info/**/*"/>
>>>               <fileset dir="${share.classes}"
>>>                        includes="**"
>>>                        excludes=".dependency-info/**/*"/>
>>>  
>>>  
>>>  
>>>  
>>>/
>>
> Sylvain.
>
>
>  
>


Re: cvs commit: incubator-myfaces/build build.xml

Posted by Sylvain Vieujot <sv...@apache.org>.
Done.
Sorry for that.

Maybe we should anyway make a package that has everything, like the
javamail distribution that has several jar files (imap.jar, pop.jar,
...) and one big mail.jar that has everything and is way easier to
install.

With our file names, a simple guy like me thinks that :
myfaces-jsf-api.jar contains the api classes.
myfaces-components.jar contains the myfaces extensions.
myfaces.jar contains everything.

So, why not have this scheme :
myfaces-jsf-api.jar contains the api classes (no change)
myfaces-impl.jar contains the standard jsf implementation, without the
extensions (same as jsf-impl.jar from the RI)
myfaces-extensions.jar contains the myfaces extensions
and a myfaces.jar that contains everything.

I think this would have the benefits of a clear modular split, with file
names similar to the RI ones, and an easy to use package for those that
don't want to mix the RI with myfaces extensions.

By the way, right now, it seems that myfaces-components.jar has the
components ressources, but that myfaces.jar doesn't has them.
Is this intended ?

Sylvain.

On Mon, 2004-11-01 at 18:43, Oliver Rossmueller wrote:

> Sylvain,
> 
> the api classes are in myfaces-jsf-api.jar now, the split was done on 
> occasion in July (see revision 1.48/1.49 of build.xml, search for 
> subject '[Myfaces-develop] JAR-Files' in the 
> myfaces-develop@lists.sourceforge.net archive)  so please revert this 
> modification.
> 
> Oliver
> 
> 
> svieujot@apache.org wrote:
> 
> >svieujot    2004/11/01 08:21:46
> >
> >  Modified:    build    build.xml
> >  Log:
> >  Add the api.classes to myfaces.jar
> >  
> >  Revision  Changes    Path
> >  1.58      +2 -2      incubator-myfaces/build/build.xml
> >  
> >  Index: build.xml
> >  ===================================================================
> >  RCS file: /home/cvs/incubator-myfaces/build/build.xml,v
> >  retrieving revision 1.57
> >  retrieving revision 1.58
> >  diff -u -r1.57 -r1.58
> >  --- build.xml	26 Oct 2004 12:41:36 -0000	1.57
> >  +++ build.xml	1 Nov 2004 16:21:46 -0000	1.58
> >  @@ -119,9 +119,9 @@
> >               </manifest>
> >   
> >               <!-- classes -->
> >  -            <!--fileset dir="${api.classes}"
> >  +            <fileset dir="${api.classes}"
> >                        includes="**"
> >  -                     excludes=".dependency-info/**/*"/-->
> >  +                     excludes=".dependency-info/**/*"/>
> >               <fileset dir="${share.classes}"
> >                        includes="**"
> >                        excludes=".dependency-info/**/*"/>
> >  
> >  
> >  
> >  
> >


Sylvain.