From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from fgw20-4.mail.saunalahti.fi (fgw20-4.mail.saunalahti.fi [62.142.5.107]) by sourceware.org (Postfix) with ESMTPS id 852063858D28 for ; Fri, 24 Mar 2023 13:31:39 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 852063858D28 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=cante.net Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=cante.net Received: from congo.cante.net (81-197-199-99.elisa-laajakaista.fi [81.197.199.99]) by fgw20.mail.saunalahti.fi (Halon) with ESMTP id 331712d6-ca48-11ed-b3cf-005056bd6ce9; Fri, 24 Mar 2023 15:31:37 +0200 (EET) Received: from jaalto by congo.cante.net with local (Exim 4.96) (envelope-from ) id 1pfhW4-004Cop-0J; Fri, 24 Mar 2023 15:31:36 +0200 Date: Fri, 24 Mar 2023 15:31:36 +0200 From: Jari Aalto To: cygwin-apps@cygwin.com Cc: CC@congo.cante.net, Brian Inglis , Doug Henderson , Adam Dinwoodie Subject: Re: xlsx2csv package may not be required. Message-ID: References: <20230317092935.gnhy6het67peihf5@lucy.dinwoodie.org> <7edd0383-15d6-4020-ef87-804f13a225c6@Shaw.ca> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable In-Reply-To: <7edd0383-15d6-4020-ef87-804f13a225c6@Shaw.ca> X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00,KAM_DMARC_STATUS,KAM_SHORT,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_PASS,TXREP,T_PDS_OTHER_BAD_TLD autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: 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=E2=80=AFAM Adam Dinwoodie via Cygwin-ap= ps > > > wrote: > > > >=20 > > > > 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. > > > > >=20 > > > > > 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. > > > >=20 > > > > Installing using pip or similar is an option for the vast majority = of > > > > packages that are available through the Cygwin installer; by that l= ogic > > > > 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! > > > >=20 > > > > 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 n= eed > > > > to even think about whether the package is a pure Python package or= not. > > >=20 > > >=20 > > > I agree with Adam. > > > I would have no problem to release the python package no if not for > > > the problem to the laptop > > >=20 > > > 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) > >=20 > > I agree, it's better to have native Python package which is more > > likely to be kept up to date by the upstream. > >=20 > > So, anyopne packaging the Python PIP version, please go ahead and > > remove the current package. >=20 > Hi Jari, >=20 > 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-dep= recated-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