Bug 2245637 - rss2email fails to build with Python 3.13: ValueError: error processing data/allthingsrss/1.config and similar failures
Summary: rss2email fails to build with Python 3.13: ValueError: error processing data/...
Keywords:
Status: CLOSED WORKSFORME
Alias: None
Product: Fedora
Classification: Fedora
Component: rss2email
Version: 40
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: David Kaufmann
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 2292003 (view as bug list)
Depends On:
Blocks: F41FTBFS F41FailsToInstall PYTHON3.13
TreeView+ depends on / blocked
 
Reported: 2023-10-23 12:01 UTC by Karolina Surma
Modified: 2024-06-12 17:55 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-06-12 17:55:44 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Karolina Surma 2023-10-23 12:01:43 UTC
rss2email fails to build with Python 3.13.0a1.

+ /usr/bin/python3 ./test/test.py
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14/./test/test.py", line 35, in <module>
    import rss2email.feed as _rss2email_feed
  File "/builddir/build/BUILD/rss2email-3.14/rss2email/feed.py", line 75, in <module>
    import feedparser as _feedparser
  File "/usr/lib/python3.13/site-packages/feedparser/__init__.py", line 28, in <module>
    from .api import parse
  File "/usr/lib/python3.13/site-packages/feedparser/api.py", line 35, in <module>
    from .encodings import convert_to_utf8
  File "/usr/lib/python3.13/site-packages/feedparser/encodings.py", line 29, in <module>
    import cgi
ModuleNotFoundError: No module named 'cgi'

According to https://docs.python.org/3.13/whatsnew/3.13.html

PEP 594: Remove the cgi and cgitb modules, deprecated in Python 3.11.

cgi.FieldStorage can typically be replaced with urllib.parse.parse_qsl() for GET and HEAD requests, and the email.message module or multipart PyPI project for POST and PUT.

cgi.parse() can be replaced by calling urllib.parse.parse_qs() directly on the desired query string, except for multipart/form-data input, which can be handled as described for cgi.parse_multipart().

cgi.parse_multipart() can be replaced with the functionality in the email package (e.g. email.message.EmailMessage and email.message.Message) which implements the same MIME RFCs, or with the multipart PyPI project.

cgi.parse_header() can be replaced with the functionality in the email package, which implements the same MIME RFCs. For example, with email.message.EmailMessage:

from email.message import EmailMessage
msg = EmailMessage()
msg['content-type'] = 'application/json; charset="utf8"'
main, params = msg.get_content_type(), msg['content-type'].params

(Contributed by Victor Stinner in gh-104773.)


https://docs.python.org/3.13/whatsnew/3.13.html

For the build logs, see:
https://copr-be.cloud.fedoraproject.org/results/@python/python3.13/fedora-rawhide-x86_64/06551235-rss2email/

For all our attempts to build rss2email with Python 3.13, see:
https://copr.fedorainfracloud.org/coprs/g/python/python3.13/package/rss2email/

Testing and mass rebuild of packages is happening in copr.
You can follow these instructions to test locally in mock if your package builds with Python 3.13:
https://copr.fedorainfracloud.org/coprs/g/python/python3.13/

Let us know here if you have any questions.

Python 3.13 is planned to be included in Fedora 41.
To make that update smoother, we're building Fedora packages with all pre-releases of Python 3.13.
A build failure prevents us from testing all dependent packages (transitive [Build]Requires),
so if this package is required a lot, it's important for us to get it fixed soon.

We'd appreciate help from the people who know this package best,
but if you don't want to work on this now, let us know so we can try to work around it on our side.

Comment 1 David Kaufmann 2023-11-23 03:05:12 UTC
this depends on a new release of feedparser (related bug: https://github.com/kurtmckee/feedparser/issues/330 ) and update of the python-feedparser rpm package

Comment 2 Aoife Moloney 2024-02-15 23:00:18 UTC
This bug appears to have been reported against 'rawhide' during the Fedora Linux 40 development cycle.
Changing version to 40.

Comment 3 hannes 2024-02-21 05:48:44 UTC
I requested it here already, perhaps we need to make a bit more pressure https://bugzilla.redhat.com/show_bug.cgi?id=2262610

Comment 4 Karolina Surma 2024-06-12 15:11:36 UTC
*** Bug 2292003 has been marked as a duplicate of this bug. ***

Comment 5 David Kaufmann 2024-06-12 15:55:10 UTC
This is was actually resolved some time ago, bug 2292003 fails because the dependencies did not rebuild, not because of this issue.
→ Closing.

Comment 6 Miro Hrončok 2024-06-12 16:00:44 UTC
It fails to install because it fails to build:

+ /usr/bin/python3 ./test/test.py
/usr/lib/python3.13/site-packages/feedparser/html.py:152: DeprecationWarning: 'count' is passed as positional argument
  data = re.sub(r'<!((?!DOCTYPE|--|\[))', r'&lt;!\1', data, re.IGNORECASE)
EEEEEEE.........................../usr/lib64/python3.13/multiprocessing/popen_fork.py:67: DeprecationWarning: This process (pid=425) is multi-threaded, use of fork() may lead to deadlocks in the child.
  self.pid = os.fork()
...
======================================================================
ERROR: test_email_data/allthingsrss/1.config (__main__.TestEmails.test_email_data/allthingsrss/1.config)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 62, in fn
    self.run_single_test(config_path=test_path)
    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 154, in run_single_test
    raise ValueError(
    ...<2 lines>...
            '\n'.join(diff_lines)))
ValueError: error processing data/allthingsrss/1.config
--- expected
+++ generated
@@ -74,7 +74,7 @@
 
 Through `r2e pause _n_` where _n_ is a feed number, you can temporarily
 suspend checking that feed for new content. To start checking it again, simply
-run `r2e unpause _n_`. When you `r2e list`, an asterisk indicates that the
+run `r2e unpause  _n_`. When you `r2e list`, an asterisk indicates that the
 feed is currently unpaused and active.
 
 [![](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/di)](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/da)  
======================================================================
ERROR: test_email_data/allthingsrss/2.config (__main__.TestEmails.test_email_data/allthingsrss/2.config)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 62, in fn
    self.run_single_test(config_path=test_path)
    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 154, in run_single_test
    raise ValueError(
    ...<2 lines>...
            '\n'.join(diff_lines)))
ValueError: error processing data/allthingsrss/2.config
--- expected
+++ generated
@@ -74,7 +74,7 @@
 
 through `r2e pause _n_` where _n_ is a feed number, you can temporarily
 suspend checking that feed for new content. to start checking it again, simply
-run `r2e unpause _n_`. when you `r2e list`, an asterisk indicates that the
+run `r2e unpause  _n_`. when you `r2e list`, an asterisk indicates that the
 feed is currently unpaused and active.
 
 [![](http://feedads.g.doubleclick.net/~a/nygtsiuss9pmvrz6092xgghnnkg/0/di)](http://feedads.g.doubleclick.net/~a/nygtsiuss9pmvrz6092xgghnnkg/0/da)  
======================================================================
ERROR: test_email_data/allthingsrss/3.config (__main__.TestEmails.test_email_data/allthingsrss/3.config)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 62, in fn
    self.run_single_test(config_path=test_path)
    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 154, in run_single_test
    raise ValueError(
    ...<2 lines>...
            '\n'.join(diff_lines)))
ValueError: error processing data/allthingsrss/3.config
--- expected
+++ generated
@@ -76,7 +76,7 @@
 
 Through `r2e pause _n_` where _n_ is a feed number, you can
 temporarily suspend checking that feed for new content. To start
-checking it again, simply run `r2e unpause _n_`. When you `r2e list`,
+checking it again, simply run `r2e unpause  _n_`. When you `r2e list`,
 an asterisk indicates that the feed is currently unpaused and active.
 
 [![](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/di)](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/da)  
======================================================================
ERROR: test_email_data/allthingsrss/4.config (__main__.TestEmails.test_email_data/allthingsrss/4.config)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 62, in fn
    self.run_single_test(config_path=test_path)
    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 154, in run_single_test
    raise ValueError(
    ...<2 lines>...
            '\n'.join(diff_lines)))
ValueError: error processing data/allthingsrss/4.config
--- expected
+++ generated
@@ -76,7 +76,7 @@
 
 Through `r2e pause _n_` where _n_ is a feed number, you can
 temporarily suspend checking that feed for new content. To start
-checking it again, simply run `r2e unpause _n_`. When you `r2e list`,
+checking it again, simply run `r2e unpause  _n_`. When you `r2e list`,
 an asterisk indicates that the feed is currently unpaused and active.
 
 [![](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/di)](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/da)  
======================================================================
ERROR: test_email_data/allthingsrss/5.config (__main__.TestEmails.test_email_data/allthingsrss/5.config)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 62, in fn
    self.run_single_test(config_path=test_path)
    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 154, in run_single_test
    raise ValueError(
    ...<2 lines>...
            '\n'.join(diff_lines)))
ValueError: error processing data/allthingsrss/5.config
--- expected
+++ generated
@@ -96,7 +96,7 @@
 
 Through `r2e pause _n_` where _n_ is a feed number, you can
 temporarily suspend checking that feed for new content. To start
-checking it again, simply run `r2e unpause _n_`. When you `r2e list`,
+checking it again, simply run `r2e unpause  _n_`. When you `r2e list`,
 an asterisk indicates that the feed is currently unpaused and active.
 
 [![][4]][5]  
======================================================================
ERROR: test_email_data/allthingsrss/6.config (__main__.TestEmails.test_email_data/allthingsrss/6.config)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 62, in fn
    self.run_single_test(config_path=test_path)
    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 154, in run_single_test
    raise ValueError(
    ...<2 lines>...
            '\n'.join(diff_lines)))
ValueError: error processing data/allthingsrss/6.config
--- expected
+++ generated
@@ -86,7 +86,7 @@
 
 Through `r2e pause _n_` where _n_ is a feed number, you can
 temporarily suspend checking that feed for new content. To start
-checking it again, simply run `r2e unpause _n_`. When you `r2e list`,
+checking it again, simply run `r2e unpause  _n_`. When you `r2e list`,
 an asterisk indicates that the feed is currently unpaused and active.
 
 [![][4]][5]  
======================================================================
ERROR: test_email_data/allthingsrss/7.config (__main__.TestEmails.test_email_data/allthingsrss/7.config)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 62, in fn
    self.run_single_test(config_path=test_path)
    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/rss2email-3.14-build/rss2email-3.14/./test/test.py", line 154, in run_single_test
    raise ValueError(
    ...<2 lines>...
            '\n'.join(diff_lines)))
ValueError: error processing data/allthingsrss/7.config
--- expected
+++ generated
@@ -74,7 +74,7 @@
 
 Through `r2e pause _n_` where _n_ is a feed number, you can temporarily
 suspend checking that feed for new content. To start checking it again, simply
-run `r2e unpause _n_`. When you `r2e list`, an asterisk indicates that the
+run `r2e unpause  _n_`. When you `r2e list`, an asterisk indicates that the
 feed is currently unpaused and active.
 
 [![](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/di)](http://feedads.g.doubleclick.net/~a/nYgTsIUsS9pmvRZ6092XGGHnNKg/0/da)  
----------------------------------------------------------------------
Ran 37 tests in 11.699s
FAILED (errors=7)

Comment 7 Fedora Fails To Install 2024-06-12 17:55:44 UTC
Hello,

Please note that this comment was generated automatically by https://pagure.io/releng/blob/main/f/scripts/ftbfs-fti/follow-policy.py
If you feel that this output has mistakes, please open an issue at https://pagure.io/releng/

All subpackages of a package against which this bug was filled are now installable or removed from Fedora 41.

Thanks for taking care of it!


Note You need to log in before you can comment on or make changes to this bug.