unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* python-cffi and ruby test suites fail in out-of-tree builds
@ 2020-05-22  1:29 Daniel Kahn Gillmor
  2020-05-23 19:52 ` Floris Bruynooghe
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Daniel Kahn Gillmor @ 2020-05-22  1:29 UTC (permalink / raw)
  To: notmuch


[-- Attachment #1.1: Type: text/plain, Size: 714 bytes --]

Hey folks--

I just did a bit of testing and cleanup for out-of-tree builds (see the
minor patches that should have landed on the list in the last hour or
two).

For me, "make check" in an out-of-tree build works fine now, with the
exception of T391-python-cffi.sh and T395-ruby.sh.

I'm afraid i don't know enough about either python or ruby to see the
obvious fix for these builds, but i do note that they're the only builds
which depend on copying their source into the out-of-tree location
(assuming no one proposes an improvement for
id:20200522010359.715688-1-dkg@fifthhorseman.net).

Just thought i'd flag this as an outstanding problem that hopefully
someone clever feels like digging into.

        --dkg

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: python-cffi and ruby test suites fail in out-of-tree builds
  2020-05-22  1:29 python-cffi and ruby test suites fail in out-of-tree builds Daniel Kahn Gillmor
@ 2020-05-23 19:52 ` Floris Bruynooghe
  2020-05-26 17:08   ` Daniel Kahn Gillmor
  2020-05-30 15:48   ` David Bremner
  2020-05-26 17:06 ` [PATCH] tests/ruby: Ensure that test works for " Daniel Kahn Gillmor
  2021-12-23 12:57 ` python-cffi and ruby test suites fail in " David Bremner
  2 siblings, 2 replies; 8+ messages in thread
From: Floris Bruynooghe @ 2020-05-23 19:52 UTC (permalink / raw)
  To: Daniel Kahn Gillmor, notmuch

Hi,

On Thu 21 May 2020 at 21:29 -0400, Daniel Kahn Gillmor wrote:

> Hey folks--
>
> I just did a bit of testing and cleanup for out-of-tree builds (see the
> minor patches that should have landed on the list in the last hour or
> two).

It probably is indeed the unfortunate case that copying the python
source is currently the easiest.  I had a quick look and it seemed like
one'd have to dig into the cffi setuptools support to make this work and
I'm not sure how successful that would be, but I admit I didn't feel
like trying.

> For me, "make check" in an out-of-tree build works fine now, with the
> exception of T391-python-cffi.sh and T395-ruby.sh.

I checked out the python one, and maybe that's not too hard.  The
following patch made this work for both in tree and out of tree (on top
of your other patch):

modified   test/T391-python-cffi.sh
@@ -8,7 +8,7 @@ fi
 
 
 test_begin_subtest "python cffi tests"
-pytest_dir=$NOTMUCH_SRCDIR/bindings/python-cffi/build/stage
+pytest_dir=$NOTMUCH_BUILDDIR/bindings/python-cffi/build/stage
 printf "[pytest]\nminversion = 3.0\naddopts = -ra\n" > $pytest_dir/pytest.ini
 test_expect_success "(cd $pytest_dir && ${NOTMUCH_PYTHON} -m pytest --log-file=$TMP_DIRECTORY/test.output)"
 test_done



Hope that's helpful.

Cheers,
Floris

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH] tests/ruby: Ensure that test works for out-of-tree builds
  2020-05-22  1:29 python-cffi and ruby test suites fail in out-of-tree builds Daniel Kahn Gillmor
  2020-05-23 19:52 ` Floris Bruynooghe
@ 2020-05-26 17:06 ` Daniel Kahn Gillmor
  2020-05-26 17:07   ` Daniel Kahn Gillmor
  2020-05-31 19:16   ` David Bremner
  2021-12-23 12:57 ` python-cffi and ruby test suites fail in " David Bremner
  2 siblings, 2 replies; 8+ messages in thread
From: Daniel Kahn Gillmor @ 2020-05-26 17:06 UTC (permalink / raw)
  To: Notmuch Mail

---
 test/test-lib.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/test-lib.sh b/test/test-lib.sh
index 792b1cb9..78a90862 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -1081,7 +1081,7 @@ test_python() {
 }
 
 test_ruby() {
-    MAIL_DIR=$MAIL_DIR $NOTMUCH_RUBY -I $NOTMUCH_SRCDIR/bindings/ruby> OUTPUT
+    MAIL_DIR=$MAIL_DIR $NOTMUCH_RUBY -I "$NOTMUCH_BUILDDIR/bindings/ruby"> OUTPUT
 }
 
 test_C () {
-- 
2.26.2

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] tests/ruby: Ensure that test works for out-of-tree builds
  2020-05-26 17:06 ` [PATCH] tests/ruby: Ensure that test works for " Daniel Kahn Gillmor
@ 2020-05-26 17:07   ` Daniel Kahn Gillmor
  2020-05-31 19:16   ` David Bremner
  1 sibling, 0 replies; 8+ messages in thread
From: Daniel Kahn Gillmor @ 2020-05-26 17:07 UTC (permalink / raw)
  To: Notmuch Mail

On Tue 2020-05-26 13:06:02 -0400, Daniel Kahn Gillmor wrote:
> ---
>  test/test-lib.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index 792b1cb9..78a90862 100644
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -1081,7 +1081,7 @@ test_python() {
>  }
>  
>  test_ruby() {
> -    MAIL_DIR=$MAIL_DIR $NOTMUCH_RUBY -I $NOTMUCH_SRCDIR/bindings/ruby> OUTPUT
> +    MAIL_DIR=$MAIL_DIR $NOTMUCH_RUBY -I "$NOTMUCH_BUILDDIR/bindings/ruby"> OUTPUT
>  }
>  
>  test_C () {

I should note that i'm not a ruby developer, but this seems to be the
correct patch to solve the problem, similar to what Floris proposed to
fix the python-cffi test suite.

    --dkg

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: python-cffi and ruby test suites fail in out-of-tree builds
  2020-05-23 19:52 ` Floris Bruynooghe
@ 2020-05-26 17:08   ` Daniel Kahn Gillmor
  2020-05-30 15:48   ` David Bremner
  1 sibling, 0 replies; 8+ messages in thread
From: Daniel Kahn Gillmor @ 2020-05-26 17:08 UTC (permalink / raw)
  To: Floris Bruynooghe, notmuch


[-- Attachment #1.1: Type: text/plain, Size: 627 bytes --]

On Sat 2020-05-23 21:52:18 +0200, Floris Bruynooghe wrote:
> modified   test/T391-python-cffi.sh
> @@ -8,7 +8,7 @@ fi
>  
>  
>  test_begin_subtest "python cffi tests"
> -pytest_dir=$NOTMUCH_SRCDIR/bindings/python-cffi/build/stage
> +pytest_dir=$NOTMUCH_BUILDDIR/bindings/python-cffi/build/stage
>  printf "[pytest]\nminversion = 3.0\naddopts = -ra\n" > $pytest_dir/pytest.ini
>  test_expect_success "(cd $pytest_dir && ${NOTMUCH_PYTHON} -m pytest --log-file=$TMP_DIRECTORY/test.output)"
>  test_done

I've tested this and i can confirm that it works for both out-of-tree
and in-tree builds.  LGTM.

     --dkg

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: python-cffi and ruby test suites fail in out-of-tree builds
  2020-05-23 19:52 ` Floris Bruynooghe
  2020-05-26 17:08   ` Daniel Kahn Gillmor
@ 2020-05-30 15:48   ` David Bremner
  1 sibling, 0 replies; 8+ messages in thread
From: David Bremner @ 2020-05-30 15:48 UTC (permalink / raw)
  To: Floris Bruynooghe, Daniel Kahn Gillmor, notmuch

Floris Bruynooghe <flub@devork.be> writes:
>
> I checked out the python one, and maybe that's not too hard.  The
> following patch made this work for both in tree and out of tree (on top
> of your other patch):
>
> modified   test/T391-python-cffi.sh
> @@ -8,7 +8,7 @@ fi
>  
>  
>  test_begin_subtest "python cffi tests"
> -pytest_dir=$NOTMUCH_SRCDIR/bindings/python-cffi/build/stage
> +pytest_dir=$NOTMUCH_BUILDDIR/bindings/python-cffi/build/stage
>  printf "[pytest]\nminversion = 3.0\naddopts = -ra\n" > $pytest_dir/pytest.ini
>  test_expect_success "(cd $pytest_dir && ${NOTMUCH_PYTHON} -m pytest --log-file=$TMP_DIRECTORY/test.output)"
>  test_done
>

I smooshed this into id:20200522010359.715688-1-dkg@fifthhorseman.net
and pushed to master.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] tests/ruby: Ensure that test works for out-of-tree builds
  2020-05-26 17:06 ` [PATCH] tests/ruby: Ensure that test works for " Daniel Kahn Gillmor
  2020-05-26 17:07   ` Daniel Kahn Gillmor
@ 2020-05-31 19:16   ` David Bremner
  1 sibling, 0 replies; 8+ messages in thread
From: David Bremner @ 2020-05-31 19:16 UTC (permalink / raw)
  To: Daniel Kahn Gillmor, Notmuch Mail

Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:

> ---
>  test/test-lib.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index 792b1cb9..78a90862 100644
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -1081,7 +1081,7 @@ test_python() {
>  }
>  
>  test_ruby() {
> -    MAIL_DIR=$MAIL_DIR $NOTMUCH_RUBY -I $NOTMUCH_SRCDIR/bindings/ruby> OUTPUT
> +    MAIL_DIR=$MAIL_DIR $NOTMUCH_RUBY -I "$NOTMUCH_BUILDDIR/bindings/ruby"> OUTPUT
>  }
>  
>  test_C () {
> -- 

pushed to master

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: python-cffi and ruby test suites fail in out-of-tree builds
  2020-05-22  1:29 python-cffi and ruby test suites fail in out-of-tree builds Daniel Kahn Gillmor
  2020-05-23 19:52 ` Floris Bruynooghe
  2020-05-26 17:06 ` [PATCH] tests/ruby: Ensure that test works for " Daniel Kahn Gillmor
@ 2021-12-23 12:57 ` David Bremner
  2 siblings, 0 replies; 8+ messages in thread
From: David Bremner @ 2021-12-23 12:57 UTC (permalink / raw)
  To: Daniel Kahn Gillmor, notmuch

Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:

> Hey folks--
>
> I just did a bit of testing and cleanup for out-of-tree builds (see the
> minor patches that should have landed on the list in the last hour or
> two).
>
> For me, "make check" in an out-of-tree build works fine now, with the
> exception of T391-python-cffi.sh and T395-ruby.sh.
>
> I'm afraid i don't know enough about either python or ruby to see the
> obvious fix for these builds, but i do note that they're the only builds
> which depend on copying their source into the out-of-tree location
> (assuming no one proposes an improvement for
> id:20200522010359.715688-1-dkg@fifthhorseman.net).
>
> Just thought i'd flag this as an outstanding problem that hopefully
> someone clever feels like digging into.

It's a bit of a continual battle to keep the out-of-tree stuff working,
but currently (cea1604a087645d07998c6986a8678b2af239322) this is fixed.

d

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2021-12-23 12:57 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-22  1:29 python-cffi and ruby test suites fail in out-of-tree builds Daniel Kahn Gillmor
2020-05-23 19:52 ` Floris Bruynooghe
2020-05-26 17:08   ` Daniel Kahn Gillmor
2020-05-30 15:48   ` David Bremner
2020-05-26 17:06 ` [PATCH] tests/ruby: Ensure that test works for " Daniel Kahn Gillmor
2020-05-26 17:07   ` Daniel Kahn Gillmor
2020-05-31 19:16   ` David Bremner
2021-12-23 12:57 ` python-cffi and ruby test suites fail in " David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

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).