From: Brian Inglis <Brian.Inglis@SystematicSw.ab.ca>
To: cygwin-apps@cygwin.com
Subject: Re: [ITA] wget
Date: Mon, 16 Nov 2020 14:35:10 -0700 [thread overview]
Message-ID: <77ae6cce-5146-d5aa-b6ed-c366b98eec46@SystematicSw.ab.ca> (raw)
In-Reply-To: <878sb1gqrg.fsf@Rainer.invalid>
On 2020-11-16 10:23, Achim Gratz wrote:
> Brian Inglis writes:
>> Thanks for feedback and help - if you could please check out the
>> latest cygport and CI jobs for any issues - I will upload upgrades if
>> I hear nothing in a few days.
>
> You should remove the files that are packaged for convenience and
> configure re-generates when you do the full autoreconf. You specify
> those in DISTCLEANFILES and cygport removes them immediately after it
> has unpacked the tar. You may need to also list those in DIFF_EXCLUDES.
> That should get rid of the spurious source patch in the package.
Thanks again for the feedback and help.
I dug up some Debian, Fedora, and OpenSuSE patches, and added them to the cygport.
OpenSuSE patches appear only to be available in their source package, or online
in their HTML page, for which I came up with a script to grab a package's patch
URLs, scrape those web pages for the patch source, and convert from HTML to
patch text. Do you know if there is any way to access OBS patches as plain text?
This build avoids autoreconf (from Eric).
It would make sense to list DIFF_EXCLUDES in DISTCLEANFILES to avoid them being
packaged in the first place, as long as that did not break the build.
So should I set DISTCLEANFILES to all the files *created* in the following log
extracts, and set DIFF_EXCLUDES to those for which patches are created, or could
more of each list be added to both?
"...
configure: creating ./config.status
config.status: creating Makefile
config.status: creating src/Makefile
config.status: creating doc/Makefile
config.status: creating util/Makefile
config.status: creating po/Makefile.in
config.status: creating tests/Makefile
config.status: creating fuzz/Makefile
config.status: creating lib/Makefile
config.status: creating testenv/Makefile
config.status: creating tests/certs/interca.conf
config.status: creating tests/certs/rootca.conf
config.status: creating src/config.h
config.status: linking
/cygdrive/c/projects/wget/wget-1.20.3-2.i686/src/wget-1.20.3/GNUmakefile to
GNUmakefile
config.status: executing depfiles commands
config.status: executing po-directories commands
config.status: creating po/POTFILES
config.status: creating po/Makefile
...
config.status: creating src/config.h
...
>>> Creating source patches
Makefile.in | 8 +-
aclocal.m4 | 58 +++---------------
configure | 147 +++++++++++++++++++++++++++++++++++++----------
doc/Makefile.in | 3
fuzz/Makefile.in | 3
lib/Makefile.in | 3
src/Makefile.in | 7 +-
src/config.h.in | 3
testenv/Makefile.in | 3
tests/Makefile.in | 12 ++-
tests/certs/interca.conf | 2
tests/certs/rootca.conf | 2
util/Makefile.in | 3
..."
--
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.]
next prev parent reply other threads:[~2020-11-16 21:35 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-08 20:05 [ITP] wget2 - modern fast parallel file and recursive website downloader Brian Inglis
2020-07-08 20:28 ` Brian Inglis
2020-07-09 18:27 ` Marco Atzeri
2020-07-09 18:50 ` Yaakov Selkowitz
2020-07-10 1:38 ` Brian Inglis
2020-07-10 5:27 ` Marco Atzeri
2020-10-30 21:30 ` Brian Inglis
2020-10-30 21:34 ` Eric Blake
2020-10-30 21:57 ` [ITA] wget Brian Inglis
2020-10-31 12:50 ` Marco Atzeri
2020-11-14 15:19 ` Achim Gratz
2020-11-14 22:15 ` Brian Inglis
2020-11-15 7:36 ` ASSI
2020-11-15 17:32 ` Brian Inglis
2020-11-16 17:23 ` Achim Gratz
2020-11-16 21:35 ` Brian Inglis [this message]
2020-11-17 19:10 ` Achim Gratz
2020-11-17 23:23 ` Brian Inglis
2020-11-18 14:23 ` ASSI
2020-11-10 5:29 ` [ITP] wget2 - please review if anyone has time Brian Inglis
2020-11-24 21:06 ` [ITP] wget2 Brian Inglis
2020-11-27 17:57 ` Hamish McIntyre-Bhatty
2020-11-27 18:28 ` Brian Inglis
2020-11-30 14:51 ` Hamish McIntyre-Bhatty
2020-11-30 18:29 ` Achim Gratz
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=77ae6cce-5146-d5aa-b6ed-c366b98eec46@SystematicSw.ab.ca \
--to=brian.inglis@systematicsw.ab.ca \
--cc=cygwin-apps@cygwin.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).