Re: Fix for portability of subversion/tests/cmdline/davautocheck.sh

Blair Zajac wrote:
> Hi Louis,
>
> To get these fixes into Subversion, I recommend:
>
> 1) Emailing this to dev@subversion.tigris.org.
> 2) Putting the [PATCH] string in the subject title.
> 3) Attaching the patch instead of including it inline, but have the
> patch end in .txt so that it's reabable in email clients.
> 4) Add a log message. See this URL on how to write a log message:
>
> http://subversion.tigris.org/hacking.html#log-messages
>
> While some of this is extra work, it'll make it easier for it to get
> into the code base, work that otherwise a committer would have to do.
>
> BTW, which platform are you on that these fixes are needed?
>
> Regards,
> Blair
>

Thank you Blair; will do.

I noticed the problem when pkgsrc found these non-portable things: (1) the "==" operator for "[" or "test" and (2) the use of
$RANDOM. Both of these items are not part of the POSIX specification, so any sh that adheres strictly to POSIX is not required to
implement these.

For example, the version of sh that is delivered with NetBSD does not support either of these, so here is a case where things would
definitely break.

Louis

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org
Received on Fri Mar 23 12:28:01 2007