Bug 2245637

Summary: rss2email fails to build with Python 3.13: ValueError: error processing data/allthingsrss/1.config and similar failures
Product: [Fedora] Fedora Reporter: Karolina Surma <ksurma>
Component: rss2emailAssignee: David Kaufmann <astra>
Status: CLOSED WORKSFORME QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 40CC: astra, fti-bugs, johannes.lips, ksurma, mhroncok
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2024-06-12 17:55:44 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2260875, 2260877, 2244836    

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!