quilc icon indicating copy to clipboard operation
quilc copied to clipboard

replace "more" with "cat" in forest deployement

Open vlimant opened this issue 6 years ago • 5 comments
trafficstars

running the installation with

./forest-sdk-2.10.0-linux-rpm.run` --accept --quiet

one still need to go and push the space bar because of lines

MS_PrintLicense() { if test x"$licensetxt" != x; then echo "$licensetxt" | more

more requires command line input in order to go through the licence. Essentially defeating the purpose of the --accept option

vlimant avatar Sep 12 '19 20:09 vlimant

Ah. Interesting. The tools we use to produce the SDK packages are not publicly available, and so I will make a note to fix this before the next SDK release. Thanks for your input, @vlimant :)

notmgsk avatar Sep 12 '19 20:09 notmgsk

This appears to be fixed in the latest version of makeself

appleby avatar Oct 28 '19 21:10 appleby

Here is the commit. Unfortunately, doesn't appear to have made it into a released version yet:

https://github.com/megastep/makeself/commit/6c852b089f843b9df60df3ce866d27979a1580da

appleby avatar Oct 28 '19 21:10 appleby

thanks for considering this. I notice https://github.com/megastep/makeself/pull/100 and https://github.com/megastep/makeself/pull/150 but the latest forest sdk under https://qcs.rigetti.com/sdk-downloads , namely forest-sdk-2.22.0-linux-rpm still require a human intervention in the installation.

MS_PrintLicense()
{
  if test x"$licensetxt" != x; then
    echo "$licensetxt" | more
    if test x"$accept" != xy; then
      while true
      do
        MS_Printf "Please type y to accept, n otherwise: "
        read yn
        if test x"$yn" = xn; then
          keep=n
          eval $finish; exit 1
          break;
        elif test x"$yn" = xy; then
          break;
        fi
      done
    fi
  fi
}

vlimant avatar Oct 28 '20 14:10 vlimant

this is because more does ask for "next page or quit"

vlimant avatar Oct 28 '20 15:10 vlimant