[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1017868: ssl-cert: Needless bash dependency



Hi,

Am 21.08.22 um 21:59 schrieb наб:
The installed make-ssl-cert depends on bash,
but doesn't really need to.

I am sorry, but I don't see any advantage here in switching away from bash. The performance advantage of dash over bash is completely irrelevant in make-ssl-cert, and bash is essential: yes.


See patch based on current Salsa HEAD, below;
it's shellcheck-clean.

The same could trivially be done for the tests,
since pretty much the only extension shellcheck picks up is local.

local is not a bash extension. Or what do you mean?

I haven't done a thorough review, but this bit is wrong:

ask_via_debconf() {
      RET=""
-    if db_settitle make-ssl-cert/title ; then
-	: # OK
-    else
+    if ! db_settitle make-ssl-cert/title ; then
  	echo "Debconf failed with error code $? $RET" >&2
  	echo "Maybe your debconf database is corrupt." >&2
  	echo "Try re-installing ssl-cert." >&2
      fi

$? will be 0 in the echo and not the return value from debconf.


Cheers,
Stefan


Reply to: