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

Add sqlitebrowser #86

Closed
wants to merge 1 commit into from

Conversation

digitalsleuth
Copy link
Contributor

In response to the request for SQLiteBrowser from SIFT Issue #599, this PR will add SQLiteBrowser and the associated repo to accomplish the install.

@grrrrr
Copy link

grrrrr commented Jun 23, 2023

I made a miscalculation in my ticket. The 3rd party repo has the same versions that ship with Ubuntu, both being from 2021. It could be simpler to use the version with Ubuntu instead of adding the repo and pgp keys

@digitalsleuth
Copy link
Contributor Author

Hi @grrrrr , the version that packs with Ubuntu Focal by default is 3.11.2, while the linuxgndu repo actually provides 3.12.2. Once we move to support for Jammy, the version which comes with Jammy is 3.12.1, which is still a minor version ahead of the default in Focal. Going forward, once we support Jammy and deprecate Focal this might change, however since the installation impact is an extremely small footprint for a newer version, I think it is worth it to keep the repo.

Thank you for your feedback, and we do appreciate you coming up with another approach.

Cheers!

@ekristen
Copy link
Contributor

ekristen commented Feb 1, 2024

@digitalsleuth can you update the package names to be sift-package-<package-name> and then add the package name to the name variable. This will prevent collisions if remnux has the same package or we pull in another state that uses that name.

@digitalsleuth
Copy link
Contributor Author

Sure thing! Working on it now.

@digitalsleuth
Copy link
Contributor Author

I'll do this for all packages and re-issue a new PR with all of them updated.

@digitalsleuth
Copy link
Contributor Author

This is done now, just testing. Once testing is finished, I'll submit that PR (for just the fixed names, not sqlitebrowser). Then once that's merged I'll submit the sqlitebrowser PR again.

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

Successfully merging this pull request may close these issues.

3 participants