<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">

<html lang="en">

<head>

<meta content="text/html; charset=utf-8" http-equiv="Content-Type">

<title>

GitLab

</title>

<style>img {

max-width: 100%; height: auto;

}

</style>

</head>

<body>

<div class="content">

<div>

<blockquote dir="auto">

<blockquote>

<p>To run the test suite bash is required and it doesn't need to be in /bin/sh (that's why env is used).

Can't see any usage of env except in one test. Maybe it is a future plan to use it (we should open an issue about it). It's a different thing, not directly related to this MR. But when at it: <code>#!/usr/sbin/env sh</code> as proposed won't start bash. It just finds sh even when not in /bin.</p>

</blockquote>

</blockquote>

<p dir="auto">We do not have any script with <code>#!/usr/sbin/env sh</code>. We have <code>#!/usr/sbin/env bash</code> in <code>cert-tests/pkcs12-utf8</code>, <code>cert-tests/certtool</code>, and <code>gnutls-cli-debug.sh</code>.</p>

<blockquote dir="auto">

<p>That's one type of user. There are several other types. And there are OSes that don't have latest GnuTLS in their repos. There are container builds, manual and automated, striving for a minimal build environment. Non of these is interested in a full-blown developer build with TLS fuzzer testing etc. All that is required is a <code>./configure && make && make check</code> - and the 'check' is just to test the basic functionality. If this can be run with a simple POSIX shell (and the busybox ash is maybe the best we can have here) just with trivial changes... well, then we should do it. If people want more - they should just install bash. we can easily document it.</p>

</blockquote>

<p dir="auto">Optional tests are a necessary evil and I do not think we should be introducing them, unless it is really necessary. It is very easy to skip them on an CI upgrade for example if the underlying OS changes some of the assumptions.</p>

<blockquote dir="auto">

<p>This MR does it (the requested changes will likely be addressed tomorrow, when I have some time). And talking about future bash-only scripts is a bit theoretical right now. Let's discuss those test when they arrive to leave the academical area and go back to practice. And BTW, I never said that requiring bash is a <em>big obstacle</em>. It's just a small annoyance.</p>

</blockquote>

<p dir="auto">Ok, I may be going into a bikeshedding area, so ok, if we can remove the bash as a dependency lets try it. But we should have easy macros to achieve the same functionality and let's remove them completely (i.e., not make some of these tests optional).</p>

</div>

</div>

<div class="footer" style="margin-top: 10px;">

<p style="font-size: small; color: #777777;">

—

<br>

Reply to this email directly or <a href="https://gitlab.com/gnutls/gnutls/merge_requests/809#note_119452744">view it on GitLab</a>.

<br>

You're receiving this email because of your account on gitlab.com.

If you'd like to receive fewer emails, you can

<a href="https://gitlab.com/sent_notifications/a68750adaaffef089a60fd68b96e3c25/unsubscribe">unsubscribe</a>

from this thread or

adjust your notification settings.

<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","action":{"@type":"ViewAction","name":"View Merge request","url":"https://gitlab.com/gnutls/gnutls/merge_requests/809#note_119452744"}}</script>

</p>

</div>

</body>

</html>