conu icon indicating copy to clipboard operation
conu copied to clipboard

six usage is redundant

Open pgajdos opened this issue 2 years ago • 4 comments

As you declare that this package is 3.6+, could you please consider drop of dependency on six please?

pgajdos avatar Mar 16 '23 10:03 pgajdos

Hi @pgajdos !

Yes, that's a good suggestion. This will also require a bit of code cleanup as well...

Do you think you can take a look at that and get rid of it? https://github.com/user-cont/conu/search?q=six&type=code

I'll take a look otherwise...

lachmanfrantisek avatar Mar 17 '23 10:03 lachmanfrantisek

Nazdar!

https://build.opensuse.org/package/view_file/devel:languages:python/python-conu/python-conu-no-six.patch

This works with 'not (integration or test_get_port_mappings or selinux or ownership)' pytest run, detailed build log can be found here:

https://build.opensuse.org/package/live_build_log/home:pgajdos:python/python-conu/openSUSE_Tumbleweed/x86_64

I am not 100% sure this is complete, though.

Petr

pgajdos avatar Mar 20 '23 10:03 pgajdos

Nazdar :)

I could apply your patch, but it'd be better if you created a pull request with the change so that

  • we can discuss the code there
  • you'll be recorded as an author in the history of the repo

jpopelka avatar Mar 20 '23 11:03 jpopelka

I think it is clear I am unsure I can do an upstream class submission, but here you are. Sure we can discuss, but I do not aspire to be author of this change if not necessary ;).

pgajdos avatar Mar 21 '23 11:03 pgajdos