unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Florian Friesdorf <flo@chaoflow.net>
To: jrollins@finestructure.net
Cc: notmuch@notmuchmail.org
Subject: Re: [PATCH] create and set temporary home directory
Date: Fri, 06 May 2011 11:09:19 +0200	[thread overview]
Message-ID: <87hb986vr4.fsf@eve.chaoflow.net> (raw)
In-Reply-To: <877har5uek.fsf@eve.chaoflow.net>

[-- Attachment #1: Type: text/plain, Size: 2134 bytes --]


Hi Jamie,

any chance of getting this patch into 0.6?

It would enable me to run the tests for the nixos package without
patching.

You could pull it from:
git://github.com/chaoflow/notmuch.git

branch: for-review/test-fixes
commit: d8842075b1ff6f008e60aced5f4e0b4d7ac79515

regards

On Mon, 18 Apr 2011 19:41:39 +0200, Florian Friesdorf <flo@chaoflow.net> wrote:
> 
> My first patch send to the list, not sure whether done properly.
> 
> In the master branch in test/emacs two tests access the build users home
> directory, so does emacs_deliver_message in the crypto branch.
> 
> I think the tests should not touch the build user's home directory. The
> patch creates a directory in the temporary test directory and sets home
> accordingly.
> 
> In case of a non-existent home directory, the tests are failing without
> this patch.
> 
> ---
>  test/test-lib.sh |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index 9581ead..43dae21 100755
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -923,6 +923,13 @@ rm -fr "$test" || {
>  	exit 1
>  }
>  
> +# A temporary home directory is needed by at least:
> +# - emacs/"Sending a message via (fake) SMTP"
> +# - emacs/"Reply within emacs"
> +# - crypto/emacs_deliver_message
> +export HOME="${TMP_DIRECTORY}/home"
> +mkdir -p "${HOME}"
> +
>  MAIL_DIR="${TMP_DIRECTORY}/mail"
>  export GNUPGHOME="${TMP_DIRECTORY}/gnupg"
>  export NOTMUCH_CONFIG="${TMP_DIRECTORY}/notmuch-config"
> -- 
> 1.7.4.4
> 
> 
> 
> 
> 
> -- 
> Florian Friesdorf <flo@chaoflow.net>
>   GPG FPR: 7A13 5EEE 1421 9FC2 108D  BAAF 38F8 99A3 0C45 F083
> Jabber/XMPP: flo@chaoflow.net
> IRC: chaoflow on freenode,ircnet,blafasel,OFTC
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

-- 
Florian Friesdorf <flo@chaoflow.net>
  GPG FPR: 7A13 5EEE 1421 9FC2 108D  BAAF 38F8 99A3 0C45 F083
Jabber/XMPP: flo@chaoflow.net
IRC: chaoflow on freenode,ircnet,blafasel,OFTC

[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]

  parent reply	other threads:[~2011-05-06  9:09 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-18 17:41 [PATCH] create and set temporary home directory Florian Friesdorf
2011-04-18 17:57 ` notmuch via nix/nixpkgs (was: Re: [PATCH] create and set temporary home directory) Florian Friesdorf
2011-05-06  9:09 ` Florian Friesdorf [this message]
2011-05-06 19:47   ` [PATCH] create and set temporary home directory Jameson Graef Rollins
2011-05-06 20:11     ` Florian Friesdorf
2011-05-06 21:06 ` Carl Worth
2011-05-06 21:56   ` Florian Friesdorf
2011-05-06 23:30   ` patch submission guide (was Re: [PATCH] create and set temporary home directory) Florian Friesdorf

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

  List information: https://notmuchmail.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87hb986vr4.fsf@eve.chaoflow.net \
    --to=flo@chaoflow.net \
    --cc=jrollins@finestructure.net \
    --cc=notmuch@notmuchmail.org \
    /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.
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).