On Dec 10, 2:12 pm, Thadeus Burgess <[email protected]> wrote:
> I see, Massmo, I checked my emails and I had sent you the wrong patch,
> it only included the fix for content-type of the admin application,
> instead of the logic that allows you to install an app with a .gz
> extension as well just in case.
>
> When I get home, I will send you this second part of the patch >.<
>
> -Thadeus

This is an excellent example of why web2py process would benefit from
the code-review cycle (extra eyes - even your own - are useful; we are
all busy, so things get missed; also the community is generally
"smarter" than any one of us, so additional eyes can be beneficial,
although limiting reviews to a small number of people has benefit).


>
>
>
> On Thu, Dec 10, 2009 at 1:51 PM, mdipierro <[email protected]> wrote:
> > Yes this has been fixed in trunk.
>
> > On Dec 10, 1:05 pm, Thadeus Burgess <[email protected]> wrote:
> >> The extension should have been fixed? I remember a making a patch for this.
>
> >> Also web2py should be allowing a .gz extension, as that was part of the 
> >> patch.
>
> >> -Thadeus
>
> >> On Thu, Dec 10, 2009 at 1:01 PM, jonfroehlich <[email protected]> 
> >> wrote:
> >> > ks to all for your responses. It looks like we should mark the
> >> > code that Jonathan found as a bug, given that web2py generated the
> >> > "*.w2p.gz" file that I us
>
> > --
>
> > You received this message because you are subscribed to the Google Groups 
> > "web2py-users" group.
> > To post to this group, send email to [email protected].
> > To unsubscribe from this group, send email to 
> > [email protected].
> > For more options, visit this group 
> > athttp://groups.google.com/group/web2py?hl=en.

--

You received this message because you are subscribed to the Google Groups 
"web2py-users" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/web2py?hl=en.


Reply via email to