public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Jari Aalto <jari.aalto@cante.net>
To: cygwin-apps@cygwin.com
Cc: CC@congo.cante.net, Brian Inglis <Brian.Inglis@shaw.ca>,
	Doug Henderson <djndnbvg@gmail.com>,
	Adam Dinwoodie <adam@dinwoodie.org>
Subject: Re: xlsx2csv package may not be required.
Date: Fri, 24 Mar 2023 15:31:36 +0200	[thread overview]
Message-ID: <ZB2mOJcTorGHrTnK@congo.cante.net> (raw)
In-Reply-To: <7edd0383-15d6-4020-ef87-804f13a225c6@Shaw.ca>

On 2023-03-22 13:13, Brian Inglis wrote:
> On 2023-03-22 01:14, Jari Aalto via Cygwin-apps wrote:
> > On 2023-03-18 17:56, marco atzeri via Cygwin-apps wrote:
> > > On Fri, Mar 17, 2023 at 10:29 AM Adam Dinwoodie via Cygwin-apps
> > > <cygwin-apps@cygwin.com> wrote:
> > > > 
> > > > On Thu, Mar 16, 2023 at 07:58:48PM -0600, Doug Henderson via Cygwin-apps wrote:
> > > > > There is a current pure python version of xlsx2csv which runs for many
> > > > > versions of Python 2 and Python 3.
> > > > > 
> > > > > It may not be necessary to provide a package for it in cygwin.
> > > > > Instead, users may install the pure python package from PYPI
> > > > > https://pypi.org/ using pip or another python package manager.
> > > > 
> > > > Installing using pip or similar is an option for the vast majority of
> > > > packages that are available through the Cygwin installer; by that logic
> > > > it wouldn't make sense to provide most of the Python packages we
> > > > provide.  Which wouldn't be an invalid strategy, but it would be a very
> > > > big change in how we handle things!
> > > > 
> > > > I think the advantage of using the Cygwin packages is a better
> > > > likelihood of packages actually being compatible with Cygwin, rather
> > > > than having some weird and unpredictable package dependency issue.  A
> > > > pure Python xlsx2csv is very unlikely to be affected by that sort of
> > > > issue, but providing it as a Cygwin package means users shouldn't need
> > > > to even think about whether the package is a pure Python package or not.
> > > 
> > > 
> > > I agree with Adam.
> > > I would have no problem to release the python package no if not for
> > > the problem to the laptop
> > > 
> > > I guess one month from now I will be able to be operative again
> > > (assuming the target supplier of the laptop https://frame.work/ will
> > > not have delivery problem)
> > 
> > I agree, it's better to have native Python package which is more
> > likely to be kept up to date by the upstream.
> > 
> > So, anyopne packaging the Python PIP version, please go ahead and
> > remove the current package.
> 
> Hi Jari,
> 
> So no issue if I adopt and package with cygport?
> It's a utility I use around tax time and sometimes elsewhen.

If you go ahead and take maintenance of the Pyhon PIP version of
xlsx2csv, I think we can retire the current xlsx2csv package as it is
not particularly active in git:

https://github.com/dilshod/xlsx2csv.git

2023-03-13 78be285 tdilshod Merge pull request #248 from NoSuck/hyperlinks-crash-string-bytes
2023-03-13 140e4cf tdilshod Merge pull request #252 from hugovk/replace-deprecated-BadZipfile
2023-01-09 3180d94 tdilshod version 0.8.1 (tag: 0.8.1)
2022-12-23 9b6c2df tdilshod include hidden rows option
2022-10-02 4beb5a8 tdilshod fix python2 support

Jari

      parent reply	other threads:[~2023-03-24 13:31 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-17  1:58 Doug Henderson
2023-03-17  9:29 ` Adam Dinwoodie
2023-03-18 16:56   ` marco atzeri
2023-03-22  7:14     ` Jari Aalto
2023-03-22 19:13       ` Brian Inglis
2023-03-22 19:41         ` Brian Inglis
2023-03-24 13:31         ` Jari Aalto [this message]

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=ZB2mOJcTorGHrTnK@congo.cante.net \
    --to=jari.aalto@cante.net \
    --cc=Brian.Inglis@shaw.ca \
    --cc=CC@congo.cante.net \
    --cc=adam@dinwoodie.org \
    --cc=cygwin-apps@cygwin.com \
    --cc=djndnbvg@gmail.com \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).