unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Tomi Ollila <tomi.ollila@iki.fi>
To: Vladimir Marek <Vladimir.Marek@Oracle.COM>,
	David Bremner <david@tethera.net>
Cc: notmuch@notmuchmail.org
Subject: Re: Re: Solaris support - missing or incompatible functions
Date: Mon, 20 May 2013 14:10:32 +0300	[thread overview]
Message-ID: <m2bo85q53r.fsf@guru.guru-group.fi> (raw)
In-Reply-To: <20130520094830.GL17166@vi64-x3-2e-prg06.cz.oracle.com>

On Mon, May 20 2013, Vladimir Marek <Vladimir.Marek@Oracle.COM> wrote:

>> >> There may be some prerequisites to running tests that you're missing,
>> >> and the test setup fails to tell you about. A pastebin of the 'make
>> >> test' output might be helpful.
>> >
>> > Ok, in that case please let me take a look at the tests first.
>> >
>> 
>> For what it's worth, I can confirm no test failures for me either on
>> linux (Debian wheezy).
>
> Thank you for looking into that. I'm still not done with fixing the test
> suite on Solaris, but form ~70 broken tests I'm at 28. Mostly it's
> because Solaris is trying to be posix correct while Linux does not.
>
> There was one surprise though:
>
> A='12\n34'
> echo $A
>
> prints two lines on Solaris bug only one line on Linux ('\n' is not
> interpreted). That breaks most of json tests as they contain '\n').

That must be some setting in (your?) bash:

$ sh -c 'echo '\''foo\nbar'\'''
foo
bar

$ bash -c 'echo '\''foo\nbar'\'''
foo\nbar

$ dash -c 'echo '\''foo\nbar'\'''
foo
bar

... after a bit digging ...

$ bash -c 'shopt -s xpg_echo; echo '\''foo\nbar'\'''
foo
bar

$ bash -c 'shopt -u xpg_echo; echo '\''foo\nbar'\'''
foo\nbar


you could try adding  shopt -u xpg_echo  into test-lib.sh

Tomi

  reply	other threads:[~2013-05-20 11:10 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-06 15:15 Solaris support - missing or incompatible functions Vladimir.Marek
2013-05-06 15:15 ` [PATCH 1/4] getpwuid: check for standards compliance (Solaris support) Vladimir.Marek
2013-05-06 15:16 ` [PATCH 2/4] asctime: " Vladimir.Marek
2013-05-06 15:16 ` [PATCH 3/4] strsep: check for availability " Vladimir.Marek
2013-05-06 15:16 ` [PATCH 4/4] timegm: add portable implementation " Vladimir.Marek
2013-05-12 20:02 ` Solaris support - missing or incompatible functions Jani Nikula
2013-05-12 20:21   ` Vladimir Marek
2013-05-12 20:45     ` Jani Nikula
2013-05-13  5:05       ` Vladimir Marek
2013-05-18 13:19         ` David Bremner
2013-05-20  9:48           ` Vladimir Marek
2013-05-20 11:10             ` Tomi Ollila [this message]
2013-05-20 13:19               ` Vladimir Marek
2013-08-16 12:05         ` Vladimir Marek
2013-08-16 12:32           ` Tomi Ollila

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=m2bo85q53r.fsf@guru.guru-group.fi \
    --to=tomi.ollila@iki.fi \
    --cc=Vladimir.Marek@Oracle.COM \
    --cc=david@tethera.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).