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

Update scripts to provide Developer Sandbox automatically #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

smarek
Copy link

@smarek smarek commented May 29, 2016

  • Added Sandbox env variable export and setting the correct permissions in update_wrapper.sh
  • GIT Ignoring common directories (latest, user-data-dir)
  • Resolving correctly BASEDIR when the run.sh is symlinked into another directory
  • Added optional lines to uncomment, to remove API keys missing warning and form of command to run chromium on standard user profile

@ghost
Copy link

ghost commented Feb 20, 2017

This doc https://chromium.googlesource.com/chromium/src/+/master/docs/linux_suid_sandbox.md
seems to say chrome_sandbox was removed(almost?), yet I still see it present on 451586/chrome-linux/

IMPORTANT NOTE: The Linux SUID sandbox is almost but not completely removed. See https://bugs.chromium.org/p/chromium/issues/detail?id=598454 This page is mostly out-of-date.
The SUID helper binary is called chrome_sandbox

Copy link

@Taeclimax Taeclimax left a comment

Choose a reason for hiding this comment

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

Good

@Nardol
Copy link

Nardol commented Mar 30, 2020

Hello,

We wish to revive this project, so we implemented your suggestion here: https://github.com/chromium-unofficial-latest-linux/chromium-latest-linux
The method for sandbox is different, chown and chmod are directly put in update.sh.
The .gitignore and getting directory even with a symlink has also taken into account, thanks.

As for API key, it is planned to let user the choice to set an API key or no, and eventually set if user wants to launch Chromium on user profile or not.
Do not esitate to let us know if you have some suggestions or make pull request.

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