all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: zimoun <zimon.toutoune@gmail.com>
To: Ben Sturmfels <ben@sturm.com.au>, 49993@debbugs.gnu.org
Subject: [bug#49993] [PATCH 00/18] Remove Python 2 packages possibly added for Python 2 MediaGoblin.
Date: Tue, 31 Aug 2021 08:10:53 +0200	[thread overview]
Message-ID: <86wno2dtzm.fsf@gmail.com> (raw)
In-Reply-To: <871r6a2zg7.fsf_-_@sturm.com.au>

Hi,

On Tue, 31 Aug 2021 at 11:08, Ben Sturmfels via Guix-patches via <guix-patches@gnu.org> wrote:

> Now that I think about it, I'm not sure that this ad-hoc approach to
> removing Python 2 deps is all that useful. Maybe a systemic approach is
> needed eg. drop all broken Python 2 packages and announce a future date
> to drop all non-broken Python 2 packages.

This is more or less the plan.  Although there is no fixed future date
to drop all the non-broken Python 2 packages. :-)

For instance #49272 remove 10 Python 2 packages and to find them:

--8<---------------cut here---------------start------------->8---
guix weather --display-missing \
     --substitute-urls="https://ci.guix.gnu.org" \
     | grep 'python2-'
--8<---------------cut here---------------end--------------->8---

then I pick only the leaf packages using “guix refresh -l”.  The
dependency graph needs some care because sometimes the graph is
rewritten.  For example,

        $ guix refresh -l python2-plastid
        No dependents other than itself: python2-plastid@0.4.8

Then, let try to locally build it in case something wrong happened on
the CI.  Note that the missing is checked on Berlin but the build uses
both (Berlin and Bordeaux):

        $ guix build python2-plastid --no-grafts

Or in some cases, it is a leaf package broken because another ’python2-’
package is broken.  Here, ’python2-pandas’.  In this case, it
depends. ;-)  Try to fix ’python2-pandas’ because:

        $ guix refresh -l python2-pandas
        Building the following 7 packages would ensure 12 dependent packages are rebuilt: fio@3.27 python2-pybedtools@0.8.2 python2-plastid@0.4.8 python2-biom-format@2.1.7 python2-warpedlmm@0.21 poretools@0.6.0-1.e426b1f ribodiff@0.2.2

or simply remove all the dependants in the same patch set.  Be careful,
the removal needs the correct order. :-)

Personally, I tend to remove leaf package brokens becaus they do not
build themselves and not because of broken dependencies.  For instance,
in this list (from Guix a9eb969):

--8<---------------cut here---------------start------------->8---
python2-biom-format-2.1.7   
python2-pybedtools-0.8.2    
python2-plastid-0.4.8       
python2-warpedlmm-0.21      
python2-scikit-learn-0.20.4 
python2-fastlmm-0.2.21      
python2-pandas-0.24.2       
python2-seaborn-0.9.1       
python2-pysnptools-0.4.11   
ptpython2-3.0.17            
python2-statsmodels-0.11.1  
--8<---------------cut here---------------end--------------->8---

I would remove only the package ’ptpython2’.  All the others depend on
’python2-pandas’ and remove ’python2-pandas’ means also remove ’fio’,
’poretools’ or ’ribodiff’.  And I am not enough qualified to have an
opinion if it is worth to fix them.  Therefore, I queue these packages
in my TODO list and then I revisit months later: if there are still
broken and no issue is open, it means they do not worth and can be
removed.

HTH.

All the best,
simon




  reply	other threads:[~2021-08-31  6:34 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-11  3:07 [bug#49993] [PATCH 00/18] Remove Python 2 packages possibly added for Python 2 MediaGoblin Ben Sturmfels via Guix-patches via
2021-08-11  3:08 ` [bug#49993] [PATCH 01/18] gnu: Remove python2-exif-read Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 02/18] gnu: Remove python2-pyld Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 03/18] gnu: Remove python2-jsonschema Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 04/18] gnu: Remove python2-unidecode Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 05/18] gnu: Remove python2-itsdangerous Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 06/18] gnu: Remove python2-feedgenerator Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 07/18] gnu: Remove python2-markdown Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 08/18] gnu: Remove python2-configobj Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 09/18] gnu: Remove python2-waitress Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 10/18] gnu: Remove python2-email-validator Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 11/18] gnu: Remove python2-py-bcrypt Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 12/18] gnu: Remove python2-oauthlib Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 13/18] gnu: Remove python2-wtforms Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 14/18] gnu: Remove python2-pastescript Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 15/18] gnu: Remove python2-openid-teams Ben Sturmfels via Guix-patches via
2021-08-11  3:08   ` [bug#49993] [PATCH 16/18] gnu: Remove python2-openid-cla Ben Sturmfels via Guix-patches via
2021-08-11  3:09   ` [bug#49993] [PATCH 17/18] gnu: Remove python2-openid Ben Sturmfels via Guix-patches via
2021-08-11  3:09   ` [bug#49993] [PATCH 18/18] gnu: Remove python2-celery Ben Sturmfels via Guix-patches via
2021-08-31  0:09 ` [bug#49993] Ben Sturmfels via Guix-patches via
2021-08-31  0:09   ` [bug#49993] [PATCH] gnu: Remove python2-translitcodec Ben Sturmfels via Guix-patches via
2021-08-31  1:08     ` [bug#49993] [PATCH 00/18] Remove Python 2 packages possibly added for Python 2 MediaGoblin Ben Sturmfels via Guix-patches via
2021-08-31  6:10       ` zimoun [this message]
2021-09-01 15:43         ` Ben Sturmfels via Guix-patches via
2021-09-02  5:30 ` [bug#49993] [v2 00/12] Remove Python 2 packages Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 01/12] gnu: Remove python2-pyld Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 02/12] gnu: Remove python2-itsdangerous Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 03/12] gnu: Remove python2-feedgenerator Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 04/12] gnu: Remove python2-configobj Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 05/12] gnu: Remove python2-email-validator Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 06/12] gnu: Remove python2-wtforms Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 07/12] gnu: Remove python2-pastescript Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 08/12] gnu: Remove python2-openid-teams Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 09/12] gnu: Remove python2-openid-cla Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 10/12] gnu: Remove python2-openid Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 11/12] gnu: Remove python2-celery Ben Sturmfels via Guix-patches via
2021-09-02  5:30   ` [bug#49993] [v2 12/12] gnu: Remove python2-translitcodec Ben Sturmfels via Guix-patches via
2023-05-14 23:46 ` bug#49993: [PATCH 00/18] Remove Python 2 packages possibly added for Python 2 MediaGoblin Vinicius Monego

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=86wno2dtzm.fsf@gmail.com \
    --to=zimon.toutoune@gmail.com \
    --cc=49993@debbugs.gnu.org \
    --cc=ben@sturm.com.au \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.