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.

Sign up for GitHub

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

Jump to bottom

Update Chromium Linux Build Instructions #233

Closed
wants to merge 4 commits into from
Closed

Update Chromium Linux Build Instructions #233

wants to merge 4 commits into from

Conversation

Raytonne
Copy link
Contributor

No description provided.

@Raytonne Raytonne mentioned this pull request Oct 11, 2023
Closed
@Raytonne
Copy link
Contributor Author

We'll test the Linux instruction tomorrow. Let's hold this PR until next week

@baentsch
Copy link
Member

We'll test the Linux instruction tomorrow. Let's hold this PR until next week

Thanks for the fast reaction -- and foreseeing my concern: Publishing something without having tested it IMO is not good practice.

Please also consider automating more: From what I saw you're asking users to add quite a bit of code that can (most likely will) be outdated pretty quickly (algorithm names and OIDs, for example). That's why we so far opted for a patch-based mechanism: That way it becomes visible automatically if something (triggered by up- and downstream changes) goes wrong and we don't lead users astray, i.e., have users "discover" when something breaks (I know this is pretty much "standard procedure" for commercial software but that does not mean that open source software needs to follow this approach :).

@Raytonne Raytonne closed this Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Raytonne @baentsch