Re: [PATCH rebase 0/2] Avoid unncessary rebases

Jon Turney writes: > But (i) we still read and extract the ImageBase out of every DLL in > the database, in case it's changed, I think that's how it should be. When you run rebase, you must consider that something changed behind your back and fix it up or the users will go mad. > and (ii)

[PATCH cygport 3/3] Don't allow SRC_URI or PATCH_URI to depend on ARCH

The values which SRC_URI and PATCH_URI evaluate to should not change depending on ARCH, as this will make the source package arch-dependent --- bin/cygport.in | 26 ++ 1 file changed, 26 insertions(+) diff --git a/bin/cygport.in b/bin/cygport.in index c3370e1..2b603d4

[PATCH cygport 2/3] Add src_patch_apply_hook

--- lib/check_funcs.cygpart | 1 + lib/src_prep.cygpart| 18 ++ 2 files changed, 19 insertions(+) diff --git a/lib/check_funcs.cygpart b/lib/check_funcs.cygpart index 961f263..224a7b1 100644 --- a/lib/check_funcs.cygpart +++ b/lib/check_funcs.cygpart @@ -687,6 +687,7 @@

[PATCH cygport 1/3] Document src_unpack_hook and src_patch_hook

Mention that these are unstable API and may be change in future. --- lib/src_prep.cygpart | 23 ++- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/lib/src_prep.cygpart b/lib/src_prep.cygpart index 1e9247a..2277ba4 100644 --- a/lib/src_prep.cygpart +++

[PATCH cygport 0/3] Don't allow source packages to differ by arch

This is where I got to, after the discussion at [1]. [1] https://cygwin.com/ml/cygwin-apps/2017-05/msg00050.html Jon Turney (3): Document src_unpack_hook and src_patch_hook Add src_patch_apply_hook Don't allow SRC_URI or PATCH_URI to depend on ARCH bin/cygport.in | 26

[PATCH cygport 0/2] Better handling of build requires (v2)

v2: BUILD_REQUIRES Jon Turney (2): Rename DEPEND to BUILD_REQUIRES Pass BUILD_REQUIRES through to .hint for source package as build-requires: bin/cygport.in | 7 +++ lib/check_funcs.cygpart | 10 ++ lib/pkg_pkg.cygpart | 5 + 3 files changed, 18

[PATCH cygport 2/2] Pass BUILD_REQUIRES through to .hint for source package as build-requires:

Converting a dependency atom to a package name with full generality requires a database of all the pathnames contained in all packages, so we don't even try to do that here. In the fullness of time, calm may have the information needed to perform that task, at which time it can start emitting

[PATCH cygport 1/2] Rename DEPEND to BUILD_REQUIRES

Somewhat clearer as to it's purposes, and pluralized for consistency with REQUIRES. Still support the previous name for backwards compatibility v2: Do backwards compatibility correctly so it works when not doing 'all' --- bin/cygport.in | 7 +++ lib/check_funcs.cygpart | 10

Re: [PATCH rebase 0/2] Avoid unncessary rebases

On 09/02/2018 11:59, Jon Turney wrote: Add some dignostics which report why a rebase is taking place. Use that information to fix some errors causing unnecessary rebases After these fixes, 'rebase -s' is relatively quick, with a warm disk cache. But (i) we still read and extract the

Re: [PATCH cygport 0/2] Better handling of build depends

On 13/02/2018 06:31, Yaakov Selkowitz wrote: On 2017-10-03 14:53, Jon Turney wrote: Jon Turney (2): Rename DEPEND to BUILD_DEPENDS Pass BUILD_DEPENDS through to .hint for source package as build-depends: Why can't this be BUILD_REQUIRES and build-requires: ? No reason at all. I

Re: [PATCH cygport] Produce obsoletes: headers in .hint files

On 13/02/2018 06:30, Yaakov Selkowitz wrote: On 2018-02-06 15:13, Jon Turney wrote: lib/pkg_pkg.cygpart | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lib/pkg_pkg.cygpart b/lib/pkg_pkg.cygpart index 0dba03d..aa6f0e8 100644 --- a/lib/pkg_pkg.cygpart +++ b/lib/pkg_pkg.cygpart