Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

scripts/setup: If the user answers "n" to installing new packages, the script tries to install them anyway. #19

Open
tcal-x opened this issue Mar 11, 2021 · 4 comments

Comments

@tcal-x
Copy link
Collaborator

tcal-x commented Mar 11, 2021

tim@tim-ubuntu:~/tcal-x/CFU-Playground$ ./scripts/setup 
Missing packages: ccache
Install [y]/n? n
Reading package lists... Done
Building dependency tree       
Reading state information... Done
Suggested packages:
  distcc | icecc
The following NEW packages will be installed:
  ccache
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 121 kB of archives.
After this operation, 369 kB of additional disk space will be used.
Get:1 http://us.archive.ubuntu.com/ubuntu focal/main amd64 ccache amd64 3.7.7-1 [121 kB]
Fetched 121 kB in 1s (221 kB/s)
Selecting previously unselected package ccache.
(Reading database ... 276510 files and directories currently installed.)
Preparing to unpack .../ccache_3.7.7-1_amd64.deb ...
Unpacking ccache (3.7.7-1) ...
Setting up ccache (3.7.7-1) ...
Updating symlinks in /usr/lib/ccache ...
Processing triggers for man-db (2.9.1-1) ...
Please install vivado
@tcal-x
Copy link
Collaborator Author

tcal-x commented Mar 11, 2021

I type "n" then return.

The variable answer inside the script is empty.

@Ayushsunny
Copy link

I would like to work on this issue

@tcal-x
Copy link
Collaborator Author

tcal-x commented Mar 15, 2021

Hi @Ayushsunny , sure, please take a look, thanks! I am concerned that this behavior might be host / terminal dependent, perhaps something related to CR vs CRLF. So it will be useful to have people on different systems try it, and also try any proposed fix (what fixes it on one system might break it on another system).

The first step would be for you to check if it works correctly on your system: if you type "n", the setup script should just exit.

@12562
Copy link

12562 commented Jan 16, 2023

I have corrected the issue in the pull-request #769

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants