Re: [ITA] libssh2
On 17.05.2021 19:11, Jon Turney wrote: On 17/05/2021 17:45, Marco Atzeri via Cygwin-apps wrote: On 17.05.2021 17:29, Jon Turney wrote: Hmmm... https://cygwin.com/git/?p=cygwin-htdocs.git;a=commitdiff;h=151c47f1889486a010954dad556a4f32add67015 ... is that what you intended to do? oh F**K, not at all I forgot to filter when using sed. I did right the first time... Can you revert it or should I ? Please go ahead. should be correct now. Regards Marco
Re: cygport test has zero exit status on failures
Jason Pyeron writes: > What is the historic rationale behind the "OR true" after the make > check? Not historic for the most part, I'd say. Cygport can also do cross-builds of packages and in those cases the tests will seldomly work (at all or at least partly) unless upstream walked the extra mile. Also, due to ATWIL syndrome and other factors, Cygwin is often not explicitly considered a target platform or (even then) treated wrongly in different ways, so even when building natively you will encounter your fair share of spurious test fails. > It seems silly to have to redefine src_test() as { > cd ${B} > make check > }, just to have a failure exit code if the test fail. It would be equally silly the other way around in a different set of circumstances. But yes, making this configurable might be useful, but this should then be done in the local configuration, not in the cygport file. Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Factory and User Sound Singles for Waldorf rackAttack: http://Synth.Stromeko.net/Downloads.html#WaldorfSounds
Re: [ITA] libssh2
On 2021-05-17 11:09, Achim Gratz wrote: Marco Atzeri via Cygwin-apps writes: https://cygwin.com/git/?p=cygwin-htdocs.git;a=commitdiff;h=151c47f1889486a010954dad556a4f32add67015 ... is that what you intended to do? Thanks Jon, good catch! oh F**K, not at all Well, we finally have a maintainer for all that stuff. ;-) I appreciate your vote of confidence or mad desperation, but on my system, I doubt I could even rebuild those packages in what remains of the year! Repology already picked those up: https://repology.org/maintainer/brian.inglis%40cygwin/feed-for-repo/cygwin so somebody needs to put those back down! ;^> -- Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada This email may be disturbing to some readers as it contains too much technical detail. Reader discretion is advised. [Data in binary units and prefixes, physical quantities in SI.]
RE: cygport test has zero exit status on failures
> -Original Message- > From: ASSI > Sent: Monday, May 10, 2021 1:06 AM > > Jason Pyeron writes: > > Cygport test is not returning a non zero exit status on failure! Is > > this expected? > > Yes, it is using 'make -k' under the hood. Again, if you need to do > something else than what the standard invocation provides, you would > generally write your own src_test() function. $ cat -n /usr/share/cygport/lib/src_test.cygpart 33 cygtest() { 34 if [ -e Makefile -o -e GNUmakefile -o -e makefile ] 35 then 36 if make -n check &> /dev/null 37 then 38 make -k check || true 39 elif make -n test &> /dev/null 40 then 41 make -k test || true 42 else 43 inform "No testsuite detected."; 44 fi 45 else 46 inform "No testsuite detected."; 47 fi 48 } What is the historic rationale behind the "OR true" after the make check? It seems silly to have to redefine src_test() as { cd ${B} make check }, just to have a failure exit code if the test fail. It seems to have come in at https://github.com/cygwin/cygport/commit/ef93794f35de38b28f473b8f14c4a9e5a4a4a5a6 and never changed since. The commit does not indicate why the exit status is suppressed. -Jason
RE: pdfgrep [ITA]
> -Original Message- > From: Marco Atzeri > Sent: Sunday, May 9, 2021 3:55 PM > > General suggestion: > - add BUILD_REQUIRES="libpcre-devel libgcrypt-devel libpoppler-cpp-devel" > > the test suite seems to need tcl so you may need to add more packages > > I am assuming that I should not include cygport in the build requires. These are the packages I install to build: asciidoc gcc-g++ libpoppler-cpp-devel libgcrypt-devel libpcre-devel dejagnu texlive-collection-latex cygport -Jason
Re: [ITA] libssh2
On 17/05/2021 17:45, Marco Atzeri via Cygwin-apps wrote: On 17.05.2021 17:29, Jon Turney wrote: Hmmm... https://cygwin.com/git/?p=cygwin-htdocs.git;a=commitdiff;h=151c47f1889486a010954dad556a4f32add67015 ... is that what you intended to do? oh F**K, not at all I forgot to filter when using sed. I did right the first time... Can you revert it or should I ? Please go ahead.
Re: [ITA] libssh2
Marco Atzeri via Cygwin-apps writes: >> Hmmm... >> https://cygwin.com/git/?p=cygwin-htdocs.git;a=commitdiff;h=151c47f1889486a010954dad556a4f32add67015 >> >> >> ... is that what you intended to do? > > > oh F**K, not at all Well, we finally have a maintainer for all that stuff. ;-) Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Wavetables for the Waldorf Blofeld: http://Synth.Stromeko.net/Downloads.html#BlofeldUserWavetables
Re: [ITA] libssh2
On 17.05.2021 17:29, Jon Turney wrote: On 17/05/2021 07:31, Marco Atzeri via Cygwin-apps wrote: On 17.05.2021 07:49, Brian Inglis wrote: On 2021-05-15 22:32, Marco Atzeri via Cygwin-apps wrote: On 15.05.2021 22:12, Brian Inglis wrote: Need to update curl with libssh2 latest 1.9.0 2019 from current 1.7.0 2016 I changed the maintainership to you Thanks Marco Could someone please also change maintainership of mingw packages: mingw64-i686-libssh2 ORPHANED (Yaakov Selkowitz) mingw64-x86_64-libssh2 ORPHANED (Yaakov Selkowitz) as I also have updates for these, needed to deploy updated libcurl mingw packages. done Hmmm... https://cygwin.com/git/?p=cygwin-htdocs.git;a=commitdiff;h=151c47f1889486a010954dad556a4f32add67015 ... is that what you intended to do? oh F**K, not at all I forgot to filter when using sed. I did right the first time... Can you revert it or should I ? Marco
Re: [ITA] libssh2
On 17/05/2021 07:31, Marco Atzeri via Cygwin-apps wrote: On 17.05.2021 07:49, Brian Inglis wrote: On 2021-05-15 22:32, Marco Atzeri via Cygwin-apps wrote: On 15.05.2021 22:12, Brian Inglis wrote: Need to update curl with libssh2 latest 1.9.0 2019 from current 1.7.0 2016 I changed the maintainership to you Thanks Marco Could someone please also change maintainership of mingw packages: mingw64-i686-libssh2 ORPHANED (Yaakov Selkowitz) mingw64-x86_64-libssh2 ORPHANED (Yaakov Selkowitz) as I also have updates for these, needed to deploy updated libcurl mingw packages. done Hmmm... https://cygwin.com/git/?p=cygwin-htdocs.git;a=commitdiff;h=151c47f1889486a010954dad556a4f32add67015 ... is that what you intended to do?
Re: calm needs to get restarted
On 16/05/2021 12:22, Marco Atzeri via Cygwin-apps wrote: On 16.05.2021 12:45, Achim Gratz wrote: Marco Atzeri via Cygwin-apps writes: On 16.05.2021 10:31, Achim Gratz wrote: Marco, please restart calm when you've fixed whatever went wrong with your latest package upload, thanks. I have not such privilege for what I know cygwin-admin shell access is required to restart it. It got restarted at noon, via cron most likely… Nope :) crossing finger does not crash again note for myself: do not try complex cancel and upload at the same time. Split in multiple steps Yeah, whilst in theory it can handle complex transactions, unfortunately they are not well tested. However, in this case, this was all my fault and I broke something with a recent update.
Re: public key
On 17/05/2021 04:59, Jason Pyeron wrote: Name: Jason Pyeron BEGIN SSH2 PUBLIC KEY Name: Jason Pyeron Adding/updating ssh key for Jason Pyeron Fingerprint: 2048 SHA256:sg25e8srBYQvvAI+cu7KUqAyLIytnxBCEIIE8dS/cDg no comment (RSA) Done.
Re: [PATCH setup] Add new option '--compact-os'
On May 13 16:42, Christian Franke wrote: > Corinna Vinschen via Cygwin-apps wrote: > > When running a shell script, certain executables (especially coreutils, > > gawk, sed, grep, find) are not so very infrequently accessed. Is this > > compression really feasible for these binaries? Did you compare shell > > script performance with non-compressed, XPRESS16K and LZX compressed > > /bin dir? > > Good point. Now I did a test with a ./configure script run after reboot: > There was significant difference with /bin/*.exe (only) uncompressed, NTFS-, > XPRESS16K- or LZX-compressed. Time was always around 23s. > > Here a read speed test with fast and slow storage and a 10+ years old > i7-2600K (4C/8T). The 256MiB test file was generated by concatenating > various EXE files. All file accesses were the first after reboot. AV > (defender) was turned off: > > > Compression MiB T1 T2 T3,T4 > == > None 256 0.69s 10.1s <0.02s > NTFS 159 1.03s 8.1s <0.02s > XPRESS4K 138 - > XPRESS8K 128 - > XPRESS16K 123 0.64s 5.4s <0.02s > LZX 97 0.79s 4.8s <0.02s > > T1,T2: Read whole file: time dd if=FILE bs=FILESIZE of=/dev/null > T3,T4: Read last byte: time dd if=FILE bs=1 skip=FILESIZE-1 of=/dev/null > > T1,T3: SATA SSD, raw read speed with dd bs=1M: ~520MB/s > T2,T4: USB3 flash drive via USB2, raw read speed: ~27MB/s > > > As expected, compression helps to improve 'virtual' read speed on slow > storage. Otherwise, it depends on storage speed, CPU speed, system load, ... > As unexpected (for me), even LZX seems to be suitable for random reads which > are done when EXE files are preloaded or paged-in. > > If the files were already cached, all read times were similar: ~0.135s for > the whole file. > > For more flexibility, I will provide a new version of the patch with > '--compact-os ALGORITHM' option. Great, thanks! Corinna