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

do_memory_split not found in raspi-config #219

Closed
jedahan opened this issue Dec 11, 2023 · 3 comments
Closed

do_memory_split not found in raspi-config #219

jedahan opened this issue Dec 11, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@jedahan
Copy link

jedahan commented Dec 11, 2023

What happened

raspi-config removed do_memory_split (not sure exactly when, but likely when bookworm was released).

During the install, it failed with the following error

Build 'camera-streamer' ... [SUCCESS]
/usr/bin/raspi-config: 3069: do_memory_split: not found
Trying to set minimum GPU Memory to 160 MB ... [SKIPPED]
	==> Tool 'raspi-config' not found ...

I am worried the install is now only partially working.

What did you expect to happen

To only run raspi-config do_memory_split when on older versions of raspberry pi OS

How to reproduce

On a fresh version of the latest raspberry pi os:

git clone https://github.com/mainsail-crew/crowsnest
cd crowsnest
sudo make install BASE_USER=$USER

Additional information

No response

@jedahan jedahan added the bug Something isn't working label Dec 11, 2023
@jedahan
Copy link
Author

jedahan commented Dec 11, 2023

Also the Tool 'raspi-config' not found is a little bit of a misleading error - raspi-config was found, and ran, but errors out.

@jedahan
Copy link
Author

jedahan commented Dec 11, 2023

if [[ "$(is_raspbian)" = "1" ]] && [[ -n "$(command -v raspi-config)" ]]; then

Adding another check for lsb_release or something else might be best. Or just making it a non-fatal error if it isn't already.

@mryel00
Copy link
Member

mryel00 commented Dec 11, 2023

You are right and I always forget to fix this line of code....
The error is non-lethal as you can see on the yellow [SKIPPED] message. So don't worry, everything should work without problems.

mryel00 added a commit to mryel00/crowsnest-dev that referenced this issue Jan 6, 2024
`do_memory_split` got removed on bookworm. Fixes mainsail-crew#219
@meteyou meteyou closed this as completed in 4feaab7 Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants