Hi,

I wondered if there are some python users on this list that can help us to fix issue 13838: http://bugs.freepascal.org/view.php?id=13838

Basically the svn2feed.py script from
http://svn.collab.net/repos/svn/trunk/tools/hook-scripts/svn2feed.py needs to be extended, so that is limits the length of a line in the log messageand wraps the rest to next line(s).

Patches are welcome. :-)

Vincent

2009/5/27 Vincent Snijders :

I wondered if there are some python users on this list that can help us to fix issue 13838: http://bugs.freepascal.org/view.php?id=13838

I made some comments (suggestions) in mantis.

Regards,
- Graeme -

_______________________________________________
fpGUI - a cross-platform Free Pascal GUI toolkit
http://opensoft.homeip.net/fpgui/

En/na Vincent Snijders ha escrit:

Try this (untested):

--- svn2feed.py.orig 2009-05-27 10:28:41.658689058 +0200 +++ svn2feed.py 2009-05-27 10:29:27.851320075 +0200 @@ -71,6 +71,7 @@
# $LastChangedRevision$

import sys
+import textwrap

# Python 2.4 is required for subprocess
if sys.version_info < (2, 4):
@@ -146,7 +147,7 @@
changed_data = proc.stdout.readlines()

desc = ("\nRevision: %s\nLog: %sModified: \n%s"
- % (revision, info_lines[3], changed_data))
+ % (revision, textwrap.fill(info_lines[3],60), changed_data))

item_dict = {
'author': info_lines[0].strip('\n'),

Bye

Hello Lazarus,

Wednesday, May 27, 2009, 10:04:28 AM, you wrote:

VS> I wondered if there are some python users on this list that can help us to fix issue VS> 13838: http://bugs.freepascal.org/view.php?id=13838 VS> Basically the svn2feed.py script from
VS> http://svn.collab.net/repos/svn/trunk/tools/hook-scripts/svn2feed.py needs to be VS> extended, so that is limits the length of a line in the log messageand wraps the VS> rest to next line(s).

One trick could be to change the CSS associated to change the "pre" tag behavior using this style:

<style type="text/css">
pre {
white-space: pre-wrap; /* css-3 */
white-space: -moz-pre-wrap !important; /* Mozilla, since 1999 */ white-space: -pre-wrap; /* Opera 4-6 */
white-space: -o-pre-wrap; /* Opera 7 */
word-wrap: break-word; /* Internet Explorer 5.5+ */ }
</style>

Graeme suggested something similar.

Paul do you know if this is possible?

Vincent

This looks rather easy. If a style sheet fix doesn't work, we will try this.

The hard part is getting a good limit, because it is fixed for all browser windows screens.

Vincent

Is this already checked ?

Marc

Paul looked at it:
http://www.hu.freepascal.org/fpcircbot/cgifpcbot?channel=lazarus-ide&fromdate=2009-05-27&todate=2009-05-27&linecount=50&fromtime=13%3A23%3A00&totime=14%3A02%3A00&sender=&msg=

Vincent

Yes, I changed our style few days ago. Thanks.

Best regards,
Paul Ishenin.