Hi Bernhard!<br><br><div class="gmail_quote">On Mon, Sep 13, 2010 at 12:41 PM, Bernhard Reiter <span dir="ltr"><<a href="mailto:bernhard@intevation.de">bernhard@intevation.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi Igor,<br>
<br>
Am Freitag, 10. September 2010 21:17:41 schrieb Igor Novikov:<br>
> Of course it's just an issue of methodology no more. So let's using<br>
> skencil-0.6/ as a formal trunk.<br>
<br>
cool with me. :)<br>
<br>
> I have tested Debian package for Skencil and found that it doesn't look<br>
> like a stable application:<br>
><br>
> <a href="http://saveimg.ru/pictures/10-09-10/a5e15398d26ba5c0850b13bc415c5857.png" target="_blank">http://saveimg.ru/pictures/10-09-10/a5e15398d26ba5c0850b13bc415c5857.png</a><br>
<br>
Version: 0.6.17-16 works nicely for me on Debian Lenny.<br>
<br>
There were a couple of issues and Daniel Baumann, the former Debian<br>
packager seems to have fixed some of the in -17:<br>
<a href="http://bugs.debian.org/cgi-bin/pkgreport.cgi?pkg=skencil;dist=unstable" target="_blank">http://bugs.debian.org/cgi-bin/pkgreport.cgi?pkg=skencil;dist=unstable</a><br>
He also wrote the package it self would be in a good state,<br>
but I haven't tried in Squeeze myself.<br></blockquote><div><br>It's not a problem solution. Patches from packagers are sometime unsuccessful.<br>I have tested Skencil 0.6.17 under Ubuntu 10.04 and 10.10 It seems the fixes are not suitable.<br>
<br>I have finished a draft of Skencil distutils build. Here is a my local SVN snapshot:<br><br><a href="http://sk1project.org/downloads/skencil/skencil-0.6.18-proposal.tar.gz">http://sk1project.org/downloads/skencil/skencil-0.6.18-proposal.tar.gz</a><br>
<br></div><span id="result_box" class="short_text"><span style="" title="">As requested by</span></span> Bernhard Herzog, I would like <span id="result_box" class="short_text"><span style="" title="">to agree on changes in the project structure.<br>
</span></span>If all the changes are suitable I will commit they step-by-step. The build is workable but <br>some additional issues should be fixed (bdist_deb command, translation build etc.)<br><br>Proposed changes resolve several issues:<br>
<br>1.Simple and stable distutils build (i.e. we can provide packages for users releasing 0.6.18<br>and such build highly simplifies packaging issues for maintainers)<br>2.Source code is separated from other files<br>3.Proposed source code structure allows launching Skencil using SVN copy <br>
4.Application becomes regular Python 'skencil' package<br>5.Soft link for Skencil launch script is substituted by small conventional python script in /usr/bin <br></div><br>As has been mentioned before the next global task is indentation. Following picture<br>
explains why:<br><br><a href="http://saveimg.ru/pictures/16-09-10/b9946b1a266990760a5fe4ab8c85723f.png">http://saveimg.ru/pictures/16-09-10/b9946b1a266990760a5fe4ab8c85723f.png</a> <br clear="all"><br>Yellow marks on right side and warning signs on left are places of mixed indentation. So a half of<br>
code lines has been involved in the problem (tkext.py is not an exception but rather a good example). <br>There is no difference between tabs or spaces but mixed indentation should be removed anyway. <br>If spaces are better for Skencil code style I will reformat the code using spaces in indents <br>
(for example 4 spaces per indent).<br><br>-- <br>Regards,<br><br>Igor Novikov<br>sK1 Project<br><a href="http://sk1project.org" target="_blank">http://sk1project.org</a><br><br><br>