[prev in list] [next in list] [prev in thread] [next in thread] 

List:       zope-dev
Subject:    Re: [Zope-dev] SVN: zope.file/branches/ulif-fix-menus/ Do menu-related configuration only if z.a.zcm
From:       Uli Fouquet <uli () gnufix ! de>
Date:       2012-05-28 9:16:39
Message-ID: 4FC34277.80904 () gnufix ! de
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Sun, 27 May 2012 21:09:44 -0400 Tres Seaver wrote:

> On 05/27/2012 07:36 PM, Ulrich Fouquet wrote:
>> Log message for revision 126504: Do menu-related configuration only if
>> z.a.zcmlfiles is available.
>>
>> Changed: U   zope.file/branches/ulif-fix-menus/CHANGES.txt U
>> zope.file/branches/ulif-fix-menus/src/zope/file/browser.zcml
> 
> Hmmm, looks like you forgot to 'svn add src/zope/file/menu.zcml'.

I was convinced I did, but apparently I didn't. It's in now.

Thanks for the hint!

The branch (ulif-fix-menus) could be reviewed now, although there is not
much to review.

I didn't manage to create a reasonable regression test for the use case,
as it seems to be very difficult and complex to remove some installed
package and make it temporarily unimportable.

If someone has an idea how to do this with not too much effort, I'd be
glad to add it. Simply changing sys.modules and sys.path seems not to be
enough.

If this test would be not crucial, I'd leave it this way and prepare a
minor release, maybe with some additional changes (the buildout.cfg is a
bit outdated and test coverage could be improved otherwise).

Best regards,

-- 
Uli


["signature.asc" (application/pgp-signature)]

_______________________________________________
Zope-Dev maillist  -  Zope-Dev@zope.org
https://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists -
 https://mail.zope.org/mailman/listinfo/zope-announce
 https://mail.zope.org/mailman/listinfo/zope )


[prev in list] [next in list] [prev in thread] [next in thread]