MoinMoin: MoinMoinChat/Logs/moin-dev/2008-08-30

2008-08-30T09:07:00  <ThomasWaldmann> moin
2008-08-30T12:53:58  <ThomasWaldmann> ugh, file leak
2008-08-30T12:57:48  <xorAxAx> moin byeongweon_t60
2008-08-30T12:58:07  <byeongweon_t60> xorAxAx: moin
2008-08-30T15:53:17  * ThomasWaldmann found the file leak
2008-08-30T16:09:38  <CIA-25> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 3794:560cdfdfffc3 1.7/MoinMoin/request/__init__.py: Fix leaking open edit-log files, refactor cleanup code in request.finish
2008-08-30T16:40:36  <CIA-25> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 3795:eb9ee3746030 1.7/docs/CHANGES: updated CHANGES
2008-08-30T16:51:04  <dreimark> moin
2008-08-30T16:51:33  <ThomasWaldmann> http://moinmo.in/MoinMoinBugs/TarfileCheckAbsolutePathFailed looks like some people (try to) use the code that was killed by Johannes in 1.8
2008-08-30T16:57:31  * dreimark never used that where is it decsibed?
2008-08-30T17:04:16  <dreimark> and trying it I get more failures shown
2008-08-30T17:08:02  <dreimark> ThomasWaldmann: they needed to hack it to get it working to the point they got the traceback.
2008-08-30T18:09:38  <CIA-25> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 3796:49f330e9831a 1.7/MoinMoin/support/tarfile.py:
2008-08-30T18:09:38  <CIA-25> tarfile: port fix from Python SVN rev 57617
2008-08-30T18:09:38  <CIA-25> TarFile.__init__() no longer fails if no name argument is passed and
2008-08-30T18:09:38  <CIA-25> the fileobj argument has no usable name attribute (e.g. StringIO).
2008-08-30T18:13:33  <ThomasWaldmann> dreimark: what do you mean?
2008-08-30T18:33:51  * ThomasWaldmann creates master18 wiki
2008-08-30T18:45:45  <dreimark> ThomasWaldmann: the GetText macro raised an exception
2008-08-30T18:45:47  <dreimark> http://paste.pocoo.org/show/83929/
2008-08-30T18:45:52  <dreimark> but now it didn't
2008-08-30T18:50:07  <dreimark> that action does not care any acl settings
2008-08-30T18:50:15  <dreimark> +on
2008-08-30T18:50:31  <dreimark> bbl
2008-08-30T19:33:48  <ThomasWaldmann> http://master18.moinmo.in/ and http://test18.moinmo.in/ are online now
2008-08-30T19:33:58  <ThomasWaldmann> both with modernized theme
2008-08-30T19:38:03  <birkenfeld> hmm, how is the theme modernized?
2008-08-30T19:38:30  <birkenfeld> ah, the icons
2008-08-30T19:38:43  <birkenfeld> but I can't help thinking of http://static.moinmo.in/logos/moindude.png as old-fashioned ^^
2008-08-30T19:40:42  <ThomasWaldmann> birkenfeld: TheSheep has done some CSS and icon styling
2008-08-30T19:41:18  <ThomasWaldmann> birkenfeld: that logo is just used so that people notice better on what wiki they work :)
2008-08-30T19:41:29  <birkenfeld> ah
2008-08-30T19:51:34  * ThomasWaldmann updates i18n pages
2008-08-30T21:28:27  <CIA-25> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4039:532865ba8334 1.8/ (5 files in 2 dirs): update i18n scripts to access master18 wiki
2008-08-30T21:28:29  <CIA-25> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4040:92c5fb24d88c 1.8/MoinMoin/i18n/ (42 files): updated i18n
2008-08-30T22:39:59  <CIA-25> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4041:4a994a297ba3 1.8/ (10 files in 5 dirs): replace cfg.user_autocreate setting by autocreate=<boolean> parameter of auth objects
2008-08-30T22:56:19  <CIA-25> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4042:0f1886933f80 1.8/wiki/config/more_samples/ (3 files): autocreate=<boolean> parameter - update sample configs
2008-08-30T23:45:06  * ThomasWaldmann thinks about reverting http://hg.moinmo.in/moin/1.8/rev/4340777f31f9
2008-08-30T23:45:33  <ThomasWaldmann> remove backup action
2008-08-30T23:45:34  <ThomasWaldmann> This code is quite unsafe and is totally incompatible with the
2008-08-30T23:45:34  <ThomasWaldmann> storage abstraction.
2008-08-30T23:46:34  <ThomasWaldmann> a) backup and restore is limited to a configurable list of (trusted) users
2008-08-30T23:47:17  <ThomasWaldmann> b) i think backup works fine, while restore might have issues due to overwriting (or not overwriting) of existing stuff
2008-08-30T23:48:06  <ThomasWaldmann> c) regarding storage abstraction: this is 1.8 main repo and there is no storage abstraction yet
2008-08-30T23:48:56  <ThomasWaldmann> any other thoughts about this?
2008-08-30T23:50:37  <xorAxAx> maybe it should rather go to contrib
2008-08-30T23:50:47  <xorAxAx> because the usefulness is rather limited
2008-08-30T23:51:10  <ThomasWaldmann> having a backup is very useful :)
2008-08-30T23:51:18  <xorAxAx> then the warning can be removed as well and people arent scared by the set of default actions shpped with moin :)
2008-08-30T23:51:29  <ThomasWaldmann> even if you need admin help to do a careful restore
2008-08-30T23:52:59  <ThomasWaldmann> (just think of a non-shell user using a wiki at a wiki hosting provider)
2008-08-30T23:53:30  <xorAxAx> a wiki hosting provider would provide backup services
2008-08-30T23:53:39  <xorAxAx> as much as he provides restore :)
2008-08-30T23:54:16  <ThomasWaldmann> but people usually feel better if they can have a backup themselves
2008-08-30T23:55:11  <xorAxAx> also the code is obviously broken
2008-08-30T23:55:17  <xorAxAx> it references backup_users
2008-08-30T23:56:27  <ThomasWaldmann> it works in 1.7
2008-08-30T23:57:46  <xorAxAx> it might also look much better if there is no restore promise at all
2008-08-30T23:58:09  <xorAxAx> because indeed, attempting to integrate restore into that approach is doomed to fail
2008-08-30T23:58:10  <ThomasWaldmann> i thought about remove the restore option
2008-08-30T23:59:12  <xorAxAx> hmm, its probably only the superuser comment that was wrong