◄ ► | Edit | Home | Admin | en de fr et | Sign in

#6177 (Issues after lxml upgrade)

by Luc Saffre · in Developers · assigned to Luc Saffre

Summary

About eight doctest failures in the book are caused by the new version of lxml. See the lxml changelog.

Children

Comments

  • 1 year, 3 months ago by Luc Saffre about #6177 (Issues after lxml upgrade) : Fixed. The truncate_comment function now also sanitizes the content. Reorganized the documentation, new page lino.utils.soup. I also wrote a blog entry BeautifulSoup and lxml.
  • 1 year, 3 months ago by Luc Saffre about #6177 (Issues after lxml upgrade) : And this is touching #5858 (Make links in text fields automatically clickable).
  • 1 year, 3 months ago by Luc Saffre about #6177 (Issues after lxml upgrade) : The test suite reveals that my little change is not yet the final solution.
  • 1 year, 3 months ago by Luc Saffre about #6177 (Issues after lxml upgrade) : I pushed a little change to lino for trying to fix this.
  • 1 year, 3 months ago by Luc Saffre about #6177 (Issues after lxml upgrade) : My suggestion: I'd write some doctest document that shows the change without lino so that we can post this as a question on the lxml mailing list. And then we require lxml<6 and wait a bit.
  • 1 year, 3 months ago by Luc Saffre about #6177 (Issues after lxml upgrade) : Maybe okay regarding the <p> tag, but there MUST be a space between the word "Try" and the preformatted text ("rm -r /").
  • 1 year, 3 months ago by Sharif Mehedi about #6177 (Issues after lxml upgrade) : I checked with the older version of lxml. For the text: "Try<pre>rm -r /</pre>and you might regret.", it used to insert a "p" tag at the beginning, like so: "<html><body><p>Try</p><pre>rm -r /</pre>and you might regret.</body></html>". And now all of a sudden it doesn't insert the "p" tag. In my opi...
  • 1 year, 3 months ago by Luc Saffre about #6177 (Issues after lxml upgrade) : Created new Ticket #6177 (Issues after lxml upgrade).