[opensuse-buildservice] HTML encoding and build log (PATCH)
Hello, I finally found my missing texts in build log. attached is a patch, which does entity encoding for the pre-formatted text in build log display (blame on you it as not already there! :-). I was not able to fix the "Refresh looses carriage return sometimes" problem. The webclient code looks correct to me. Maybe the problem is in website and API interaction (e.g. the return gets stripped when removing the HTML headers). Ciao -- http://www.dstoecker.eu/ (PGP key available)
On Wed, 14 Feb 2007, Dirk Stoecker wrote:
attached is a patch, which does entity encoding for the pre-formatted text in build log display (blame on you it as not already there! :-).
Something wrong with this patch or only overlooked? Ciao -- http://www.dstoecker.eu/ (PGP key available) --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org
Hi Dirk,
attached is a patch, which does entity encoding for the pre-formatted text in build log display (blame on you it as not already there! :-).
Something wrong with this patch or only overlooked?
now applied (rev. 1263). sorry for the late reply, I just overlooked your mail... -- David Mayr, http://davey.de openSUSE LINUX, http://opensuse.de --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org
On Fri, 16 Feb 2007, David Mayr wrote:
attached is a patch, which does entity encoding for the pre-formatted text in build log display (blame on you it as not already there! :-).
Something wrong with this patch or only overlooked?
now applied (rev. 1263). sorry for the late reply, I just overlooked your mail...
No problem. It's normal on mailinglists. Therefore we have bug and patch trackers. Thought mailinglist is usually easier. As weekend comes and I want to cleanup my week's work I attached two other changes in my working directory: buildlog.diff - Changes the JavaScript code on the live buildlog a bit, so now Start/Stop button appear when necessary and the buttons are also below the text. Now clicking on autoscroll+refresh below the text is possible. Would be nice to have both in one button, but I don't know how to do this. https.diff: - There are still some http accesses in the code. I changed them to https and found no negative effects (but also no positive yet). I think the http://'s are wrong, but leave it to you :-) Ciao -- http://www.dstoecker.eu/ (PGP key available)
Hi Dirk,
As weekend comes and I want to cleanup my week's work I attached two other changes in my working directory:
buildlog.diff - Changes the JavaScript code on the live buildlog a bit, so now Start/Stop button appear when necessary and the buttons are also below the text. Now clicking on autoscroll+refresh below the text is possible. Would be nice to have both in one button, but I don't know how to do this.
applied in SVN revision 1267.
https.diff: - There are still some http accesses in the code. I changed them to https and found no negative effects (but also no positive yet). I think the http://'s are wrong, but leave it to you :-)
applied in SVN revision 1266. Thanks again for your contribution! -- David Mayr, http://davey.de openSUSE LINUX, http://opensuse.de --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org
participants (2)
-
David Mayr -
Dirk Stoecker