About deploying from WARs and issue 3457

Hi dear developers of OpenLaszlo,

Did you plan to fix issue 3457 ( 
http://www.openlaszlo.org/jira/browse/LPP-3457 ) ? if yes what is the 
current timeframe ?
I've looked a little bit into the problems involved in resolving this 
issue, and this seems to be a huge work! but it would be nice to have it 
resolved.
Best regards,

--

-- 
Cordialement,
Ludo - http://www.ubik-products.com
---
"L'amour pour principe et l'ordre pour base; le progres pour but" (A.Comte) 

For Review: Change 20071001-hqm-8 Summary: only show diagnostic info in lzpix app when release is labeled 'internal'

Change 20071001-hqm-8 by hqm <at> IBM-2E06404CB67 on 2007-10-01 10:47:33 EDT
    in /cygdrive/c/users/hqm/openlaszlo/trunk2/demos
    for http://svn.openlaszlo.org/openlaszlo/trunk/demos

Summary: only show diagnostic info in lzpix app when release is
labeled 'internal'

New Features:

Bugs Fixed: LPP-4779

Technical Reviewer: ptw
QA Reviewer: josh
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:

Tests:

run lzpix

Files:
M      lzpix/app.lzx

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20071001-hqm-8.tar

svn users

Looking at the list of svn committers, I see some names I'm not  
familiar with. If you are one of these people, or know who they are,  
please ask them to contact me. Otherwise I'm going to delete them:

Biayna
shivak
seanwhe

Thanks.

-ben

For Review: Change 20071001-hqm-0 Summary: allow dhtml as default runtime

Change 20071001-hqm-0 by hqm <at> IBM-2E06404CB67 on 2007-10-01 13:24:07 EDT
    in /cygdrive/c/users/hqm/openlaszlo/trunk2
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary:  allow dhtml as default runtime

New Features:

Bugs Fixed: LPP-3705

Technical Reviewer: pbr
QA Reviewer: ptw
Doc Reviewer: jrs

Documentation:

Release Notes:

Details:

Assign default runtime from one place (LPS.java), and assume runtime
default if no 'lzr'
query arg is supplied in RESPONDERObject.java for lzt=object requests

Tests:

restart tomcat, and see if apps come up as DHTML by default with no
lzr arg, and with lzt=html queries

Files:

Re: For Review: Change 20071001-hqm-0 Summary: allow dhtml as default runtime

AWESOME! This has been on many people's wish lists for a long time.

On Oct 1, 2007, at 10:28 AM, Henry Minsky wrote:

> Change 20071001-hqm-0 by hqm <at> IBM-2E06404CB67 on 2007-10-01 13:24:07  
> EDT
>     in /cygdrive/c/users/hqm/openlaszlo/trunk2
>     for http://svn.openlaszlo.org/openlaszlo/trunk
>
> Summary:  allow dhtml as default runtime
>
> New Features:
>
> Bugs Fixed: LPP-3705
>
> Technical Reviewer: pbr
> QA Reviewer: ptw
> Doc Reviewer: jrs
>
> Documentation:
>
> Release Notes:
>
> Details:
>
> Assign default runtime from one place (LPS.java), and assume runtime
> default if no 'lzr'
> query arg is supplied in RESPONDERObject.java for lzt=object requests
>

Re: For Review: Change 20071001-hqm-0 Summary: allow dhtml as default runtime

Re: For Review: Change 20071001-hqm-8 Summary: only show diagnostic info in lzpix app when release is labeled 'internal'

Approved in spirit, but I think you need to check with Mamye what the  
possible set of values is for lpsrelease.  For instance, I think we  
want to have the test be `canvas.lpsrelease != "Production"`.

On 2007-10-01, at 10:49 EDT, Henry Minsky wrote:

> Change 20071001-hqm-8 by hqm <at> IBM-2E06404CB67 on 2007-10-01 10:47:33  
> EDT
>     in /cygdrive/c/users/hqm/openlaszlo/trunk2/demos
>     for http://svn.openlaszlo.org/openlaszlo/trunk/demos
>
> Summary: only show diagnostic info in lzpix app when release is
> labeled 'internal'
>
> New Features:
>
> Bugs Fixed: LPP-4779
>
> Technical Reviewer: ptw
> QA Reviewer: josh
> Doc Reviewer: (pending)
>
> Documentation:
>
> Release Notes:
>
> Details:
>
>
> Tests:
>
> run lzpix
>
> Files:
> M      lzpix/app.lzx
>
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20071001- 
> hqm-8.tar
>
>
> -- 
> Henry Minsky
> Software Architect
> hminsky <at> laszlosystems.com

Re: For Review: Change 20071001-hqm-8 Summary: only show diagnostic info in lzpix app when release is labeled 'internal'

[Re-sent with Josh's address corrected]

Approved in spirit, but I think you need to check with Mamye what the  
possible set of values is for lpsrelease.  For instance, I think we  
want to have the test be `canvas.lpsrelease != "Production"`.

On 2007-10-01, at 10:49 EDT, Henry Minsky wrote:

> Change 20071001-hqm-8 by hqm <at> IBM-2E06404CB67 on 2007-10-01 10:47:33  
> EDT
>     in /cygdrive/c/users/hqm/openlaszlo/trunk2/demos
>     for http://svn.openlaszlo.org/openlaszlo/trunk/demos
>
> Summary: only show diagnostic info in lzpix app when release is
> labeled 'internal'
>
> New Features:
>
> Bugs Fixed: LPP-4779
>
> Technical Reviewer: ptw
> QA Reviewer: josh
> Doc Reviewer: (pending)
>
> Documentation:
>
> Release Notes:
>
> Details:
>
>
> Tests:
>
> run lzpix
>
> Files:
> M      lzpix/app.lzx
>
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20071001- 
> hqm-8.tar
>
>
> -- 
> Henry Minsky
> Software Architect
> hminsky <at> laszlosystems.com

Re: About deploying from WARs and issue 3457

I spent quite a while investigating this, and you are correct, it  
would be a lot of work to fix.  It is unlikely that it will be fixed  
unless someone volunteers time or funding to address it.

It is difficult to imagine anyone really needs this fixed since  
nearly all container servers have an option to unpack .wars when they  
are installed.

On 2007-10-01, at 09:57 EDT, Ludovic Maitre wrote:

> Hi dear developers of OpenLaszlo,
>
> Did you plan to fix issue 3457 ( http://www.openlaszlo.org/jira/ 
> browse/LPP-3457 ) ? if yes what is the current timeframe ?
> I've looked a little bit into the problems involved in resolving  
> this issue, and this seems to be a huge work! but it would be nice  
> to have it resolved.
> Best regards,
>
> -- 
> Cordialement,
> Ludo - http://www.ubik-products.com
> ---
> "L'amour pour principe et l'ordre pour base; le progres pour  
> but" (A.Comte)

For Review: Change 20071001-ptw-m Summary: Remove old .make trampolines

Change 20071001-ptw-m by ptw <at> dueling-banjos.local on 2007-10-01  
19:55:05 EDT
     in /Users/ptw/OpenLaszlo/ringding-2
     for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Remove old .make trampolines

Bugs Fixed:
LPP-4181 'Remove vestigial .make methods from built-ins'

Technical Reviewer: hminsky (pending)
QA Reviewer: promanik (pending)

Details:
     We decided we don't need to rewrite all new calls to .make calls,
     so these old trampolines can be removed.

Tests:
     smokecheck in SWF and DHTML, calendar, amazon

Files:
M      WEB-INF/lps/lfc/core/LzNode.lzs
M      WEB-INF/lps/lfc/compiler/LzRuntime.lzs
M      WEB-INF/lps/lfc/compiler/Class.lzs

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20071001-ptw- 
m.tar