Re: Tapestry 5.3.4-rc-9

2012-07-05 Thread Howard Lewis Ship
At this point, I'm thinking 5.3.5 will fix yuicompressor. I just want
to get 5.3.4 out the door

On Wed, Jul 4, 2012 at 6:51 AM, Jochen Berger  wrote:
> Am 04.07.2012 10:55, schrieb Lenny Primak:
>
>> I don't think there is a 'clean' solution.
>
>
> Hopefully, requirejs/uglifyjs will be able to make all this obsolete by the
> end of this year. :-)
>
>
>
> -
> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> For additional commands, e-mail: users-h...@tapestry.apache.org
>



-- 
Howard M. Lewis Ship

Creator of Apache Tapestry

The source for Tapestry training, mentoring and support. Contact me to
learn how I can get you up and productive in Tapestry fast!

(971) 678-5210
http://howardlewisship.com

-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org



Re: Tapestry 5.3.4-rc-9

2012-07-04 Thread Jochen Berger

Hi,

Am 05.07.2012 02:39, schrieb Bob Harner:

I'm trying to follow what was done with YUICompressor in Tapestry
5.3.4-rc-9, somebody tell me if I have it right: To get YUICompressor
to work more reliably, Tapestry 5.3.4-rc-9 includes a version of the
YUICompressor that the maven-play-plugin project built (at
http://maven-play-plugin.googlecode.com/svn/mavenrepo/releases/),
which includes versions of Rhino classes


correct up to here AFAICT


that have had their package names changed to avoid conflicts with other 
implementations of Rhino
that may be on the classpath. Right?


No, unfortunately not. That is what would have solved the problems.
5.3.3 included rhino and yuicompressor 2.4.6. The latter contained 
modified versions of some of the classes contained in rhino, so the 
classes ended up on the classpath twice. once from rhino (unmodified) 
and once from yuicompressor (modified). This caused serious issues so 
for 5.3.4, yuicompressor was upgraded.
In 5.3.4, t-yuicompressor has a dependency on the aforementioned 
yuicompressor 2.4.7 from the maven-play repo. The only difference apart 
from the version is that that jar is a merged package of rhino, jargs 
and yuicompressor itself. No chances in package names or whatsoever, 
just all three jars merged together in one with the modified rhino 
classes from yuicompressor simply overwriting the original ones.
While this eliminates the duplication in classes between rhino and yui, 
conflicts with other libraries that depend on rhino themselves (like 
t5conduit) remain the same. Sooner or later we might even have a 
conflicting rhino in tapestry-core if Howard decides to pull it in via 
wro4j as a compile dependency for Tapestry 5.4.


Jochen

-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org



Re: Tapestry 5.3.4-rc-9

2012-07-04 Thread Lenny Primak
Sounds reasonable. I don't know what play project did though. 



On Jul 4, 2012, at 8:39 PM, Bob Harner  wrote:

> I'm trying to follow what was done with YUICompressor in Tapestry
> 5.3.4-rc-9, somebody tell me if I have it right: To get YUICompressor
> to work more reliably, Tapestry 5.3.4-rc-9 includes a version of the
> YUICompressor that the maven-play-plugin project built (at
> http://maven-play-plugin.googlecode.com/svn/mavenrepo/releases/),
> which includes versions of Rhino classes that have had their package
> names changed to avoid conflicts with other implementations of Rhino
> that may be on the classpath. Right?
> 
> On Wed, Jul 4, 2012 at 9:51 AM, Jochen Berger  
> wrote:
>> Am 04.07.2012 10:55, schrieb Lenny Primak:
>> 
>>> I don't think there is a 'clean' solution.
>> 
>> 
>> Hopefully, requirejs/uglifyjs will be able to make all this obsolete by the
>> end of this year. :-)
>> 
>> 
>> 
>> -
>> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
>> For additional commands, e-mail: users-h...@tapestry.apache.org
>> 
> 
> -
> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> For additional commands, e-mail: users-h...@tapestry.apache.org
> 

-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org



Re: Tapestry 5.3.4-rc-9

2012-07-04 Thread Bob Harner
I'm trying to follow what was done with YUICompressor in Tapestry
5.3.4-rc-9, somebody tell me if I have it right: To get YUICompressor
to work more reliably, Tapestry 5.3.4-rc-9 includes a version of the
YUICompressor that the maven-play-plugin project built (at
http://maven-play-plugin.googlecode.com/svn/mavenrepo/releases/),
which includes versions of Rhino classes that have had their package
names changed to avoid conflicts with other implementations of Rhino
that may be on the classpath. Right?

On Wed, Jul 4, 2012 at 9:51 AM, Jochen Berger  wrote:
> Am 04.07.2012 10:55, schrieb Lenny Primak:
>
>> I don't think there is a 'clean' solution.
>
>
> Hopefully, requirejs/uglifyjs will be able to make all this obsolete by the
> end of this year. :-)
>
>
>
> -
> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> For additional commands, e-mail: users-h...@tapestry.apache.org
>

-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org



Re: Tapestry 5.3.4-rc-9

2012-07-04 Thread Jochen Berger

Am 04.07.2012 10:55, schrieb Lenny Primak:

I don't think there is a 'clean' solution.


Hopefully, requirejs/uglifyjs will be able to make all this obsolete by 
the end of this year. :-)



-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org



Re: Tapestry 5.3.4-rc-9

2012-07-04 Thread Lenny Primak
I have solved this problem by repackaging
yuicompressor, rhino and tap-yuicompressor into a single .jar file
using jarjar, which solves all problems.
I don't think there is a 'clean' solution.

---
Combine yucompressor-2.4.6 and rhino-1.6R7 into src file (via jar)
$ mkdir work; cd work
$ jar xf ~/.m2/repository/rhino/js/1.6R7/js-1.6R7.jar
$ jar xf 
~/.m2/repository/com/yahoo/platform/yui/yuicompressor/2.4.6/yuicompressor-2.4.6.jar
 
$ jar xf 
~/.m2/repository/org/apache/tapestry/tapestry-yuicompressor/5.3.2/tapestry-yuicompressor-5.3.2.jar
$ jar cfm ../yuic.jar META-INF/MANIFEST.MF .

Using instructions in 
http://yuilibrary.com/projects/yuicompressor/ticket/2527981
to generate yuicompressor and upload it to the BAW repo
$ java -jar jarjar-1.2.jar process yui-rules.txt yuic.jar 
yuicompressor-2.4.6-cust-tap-5.3.2.jar

$ mvn deploy:deploy-file -DgroupId=com.yahoo.platform.yui 
-DartifactId=yuicompressor -Dversion=2.4.6-cust-tap-5.3.2 -DgeneratePom=true 
-Dpackaging=jar -Dfile=yuicompressor-2.4.6-cust-tap-5.3.2.jar 
-DrepositoryId=repo -Durl=file://flowlogix-m2
-

- yui-rules.txt --
rule jargs.gnu.** com.yahoo.platform.yui.@0 
rule org.mozilla.** com.yahoo.platform.yui.@0 
keep com.yahoo.** 
keep org.apache.**
---

On Jul 4, 2012, at 4:48 AM, Felix Gonschorek wrote:

> howard,
> 
> i encountered problems with yuicompressor too (in the rc5 release i
> think). i deactivated the yuicompressor tap5 module to fix it. the
> problem for me was this:
> 
> http://yuilibrary.com/projects/yuicompressor/ticket/2528056
> 
> the yuicompressor has its own version of the rhino classes, but in the
> same package hierachy. i have my own (newer)  rhino dependency in the
> classpath. when the yuicompressor rhino classes gets loaded first, my
> application starts to massively throw exceptions.
> 
> maybe this helps narrowing down things.
> 
> felix
> 
> 
> Am 03.07.2012 18:16, schrieb Howard Lewis Ship:
>> I've just released Tapestry 5.3.4-rc-9.
>> 
>> The only difference from rc-8 is that the POM for
>> tapestry-yuicompressor correctly identifies the repository one of its
>> dependencies comes from (that is not in the Maven central repository).
>> 
>> However, may people (including myself) are having issues with
>> yuicompressor in running applications; I'm still trying to get to the
>> core of it, my working theory is some conflict between the patched
>> version of Rhino (the JavaScript engine) inside the yuicompressor
>> library, and the version of Rhino that is packaged with JDK 1.6 and
>> up.
>> 
> 
> 
> -
> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> For additional commands, e-mail: users-h...@tapestry.apache.org
> 


-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org



Re: Tapestry 5.3.4-rc-9

2012-07-04 Thread Felix Gonschorek
howard,

i encountered problems with yuicompressor too (in the rc5 release i
think). i deactivated the yuicompressor tap5 module to fix it. the
problem for me was this:

http://yuilibrary.com/projects/yuicompressor/ticket/2528056

the yuicompressor has its own version of the rhino classes, but in the
same package hierachy. i have my own (newer)  rhino dependency in the
classpath. when the yuicompressor rhino classes gets loaded first, my
application starts to massively throw exceptions.

maybe this helps narrowing down things.

felix


Am 03.07.2012 18:16, schrieb Howard Lewis Ship:
> I've just released Tapestry 5.3.4-rc-9.
>
> The only difference from rc-8 is that the POM for
> tapestry-yuicompressor correctly identifies the repository one of its
> dependencies comes from (that is not in the Maven central repository).
>
> However, may people (including myself) are having issues with
> yuicompressor in running applications; I'm still trying to get to the
> core of it, my working theory is some conflict between the patched
> version of Rhino (the JavaScript engine) inside the yuicompressor
> library, and the version of Rhino that is packaged with JDK 1.6 and
> up.
>


-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org



Tapestry 5.3.4-rc-9

2012-07-03 Thread Howard Lewis Ship
I've just released Tapestry 5.3.4-rc-9.

The only difference from rc-8 is that the POM for
tapestry-yuicompressor correctly identifies the repository one of its
dependencies comes from (that is not in the Maven central repository).

However, may people (including myself) are having issues with
yuicompressor in running applications; I'm still trying to get to the
core of it, my working theory is some conflict between the patched
version of Rhino (the JavaScript engine) inside the yuicompressor
library, and the version of Rhino that is packaged with JDK 1.6 and
up.

-- 
Howard M. Lewis Ship

Creator of Apache Tapestry

The source for Tapestry training, mentoring and support. Contact me to
learn how I can get you up and productive in Tapestry fast!

(971) 678-5210
http://howardlewisship.com

-
To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
For additional commands, e-mail: users-h...@tapestry.apache.org