Skip to content

Conversation

0xr3act0r
Copy link
Contributor

The linux-modules-extra package must be installed to use the ubi and nandsim modules. Otherwise, nandsim and ubi modules cannot be loaded with modprobe during the dependency check phase. (Ubuntu 22.04)

Added the "linux-modules-extra".
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for contributing a pull request!

Welcome to the EMBA firmware analysis community!

We are glad you are here and appreciate your contribution. Please keep in mind our contributing guidelines here and here.
Also, please check existing open issues and consider to open a discussion in the dedicated discussion area.
Additionally, we have collected a lot of details around EMBA, the installation and the usage of EMBA in our Wiki.

If you like EMBA you have the chance to support us by becoming a Sponsor or buying some beer here.

To show your love for EMBA with nice shirts or other merch you can check our Spreadshop.

This is an automatic message. Allow for time for the EMBA community to be able to read the pull request and comment on it.

@m-1-k-3 m-1-k-3 added bug Something isn't working Ubuntu EMBA labels Nov 15, 2024
@m-1-k-3
Copy link
Member

m-1-k-3 commented Nov 15, 2024

Thank you very much for this valuable fix.
Will it also work if we move this fix to the ubuntu installer module over here?

print_tool_info "dbus-x11" 1

@m-1-k-3 m-1-k-3 self-requested a review November 15, 2024 07:21
@0xr3act0r
Copy link
Contributor Author

Thank you very much for this valuable fix. Will it also work if we move this fix to the ubuntu installer module over here?

print_tool_info "dbus-x11" 1

Of course, it might make more sense. My commit was a bit of a quick fix :-)

@m-1-k-3
Copy link
Member

m-1-k-3 commented Nov 17, 2024

Thank you for your updates. Two further points:

  • please check the shellcheck results
    image
  • If we can install it via "print_tool_info" (which will install it afterwards here
    apt-get install "${INSTALL_APP_LIST[@]}" -y --no-install-recommends
    ) we can use the typical installation mechanism and further checks already available in EMBA. Would this also work?

@0xr3act0r
Copy link
Contributor Author

I hope it is appropriate now, thank you very much for your valuable feedback :)

fixed interesting space bug in newline, there should be no more errors in the output of check-project.sh.
@m-1-k-3
Copy link
Member

m-1-k-3 commented Nov 18, 2024

Thank you for effort and bringing this fix to EMBA

@m-1-k-3 m-1-k-3 merged commit 8e98261 into e-m-b-a:master Nov 18, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working EMBA Ubuntu
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants