Fix the production dockerfile #12

blek · 2023-10-31T16:35:48+01:00

blek commented

2023-10-31 16:35:48 +01:00

There are a few problems with the current ones:

I have no idea why the current build script worked since its incompatible with the uglify-js API
Build script requires git, but the current dockerfile does not install it
For whatever reason it uses busybox:musl base image, which doesnt have a package manager, therefore would be a pain to maintain.
Build script also required .git directory to fetch commit and branch data, but it is not copied during the build step

There are a few problems with the current ones: 1. I have no idea why the current build script worked since its incompatible with the `uglify-js` API 2. Build script requires git, but the current dockerfile does not install it 3. For whatever reason it uses `busybox:musl` base image, which doesnt have a package manager, therefore would be a pain to maintain. 4. Build script also required .git directory to fetch commit and branch data, but it is not copied during the build step

blek added 3 commits 2023-10-31 16:35:49 +01:00

b1463519d7

fix prod dockerfile build error

6302d11acd

fix filed dockerfile and use alpine as a base image

6120ce7a30

get rid of function panics in the build.rs

blek merged commit 6120ce7a30 into 0.2-dev

2023-10-31 16:35:59 +01:00

blek deleted branch fix-prod-build

2023-10-31 16:36:03 +01:00

Sign in to join this conversation.

No reviewers

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blek/bfile#12