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

Remove dependencies for building from scratch #3514

Open
mathemancer opened this issue Mar 26, 2024 · 1 comment
Open

Remove dependencies for building from scratch #3514

mathemancer opened this issue Mar 26, 2024 · 1 comment
Labels
needs: requirements The problem is clear and worth solving, but we're not yet sure of the best solution type: enhancement New feature or request work: ci Related to our GitHub actions, continuous integration, and automated testing work: installation work: release
Milestone
Beta

Comments

@mathemancer
Copy link
Contributor

Problem

Our 0.1.6 instructions to build from scratch LINK HERE WHEN PUBLISHED require wget, unzip, and gettext dependencies.

Proposed solution

We should:

  • discuss whether we want to go through the effort to remove these,
  • Add whatever modifications are needed to our building and release process, and
  • remove dependencies as per the earlier bullet points.

Additional context

Based on my comments on #3494 .
@mathemancer mathemancer added type: enhancement New feature or request needs: triage This issue has not yet been reviewed by a maintainer needs: requirements The problem is clear and worth solving, but we're not yet sure of the best solution work: installation work: ci Related to our GitHub actions, continuous integration, and automated testing work: release and removed needs: triage This issue has not yet been reviewed by a maintainer labels Mar 26, 2024
@kgodey kgodey added this to the v0.1.7 milestone Mar 26, 2024
@mathemancer mathemancer added needs: product approval It's not yet clear that this issue will actually improve Mathesar from a user's perspective and removed needs: product approval It's not yet clear that this issue will actually improve Mathesar from a user's perspective labels Mar 26, 2024
@mathemancer mathemancer mentioned this issue Mar 26, 2024
Merged
@pavish
Copy link
Member

pavish commented Mar 27, 2024

Considering that our target users for the build from scratch setup are sysadmins, I'm not sure that we need to put in the additional effort at removing the wget and unzip dependencies.

I do think there might be some value in removing gettext and downloading compiled translation files similar to how we now download the frontend static files.

@kgodey kgodey modified the milestones: v0.1.7, Beta Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs: requirements The problem is clear and worth solving, but we're not yet sure of the best solution type: enhancement New feature or request work: ci Related to our GitHub actions, continuous integration, and automated testing work: installation work: release
Projects
None yet
Milestone
Beta
Development

No branches or pull requests
3 participants
@kgodey @mathemancer @pavish