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

ci: merge staging to master #94

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

ci: merge staging to master #94

wants to merge 8 commits into from

Conversation

MatrixAI-Bot
Copy link
Member

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

@MatrixAI-Bot MatrixAI-Bot self-assigned this Jun 26, 2023
@MatrixAI-Bot
Copy link
Member Author

@ghost
Copy link

ghost commented Jun 26, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@MatrixAI-Bot
Copy link
Member Author

@CMCDragonkai
Copy link
Member

Due to https://github.com/vercel/pkg-fetch/issues/285 v18.15.0 of macos-arm64 build does not exist.

Also vercel isn't really maintaining that anymore, they probably moved onto to other things. I believe we previously saw another project that can bundle binaries.

For now I've disabled the macos-arm64 build.

@MatrixAI-Bot
Copy link
Member Author

@MatrixAI-Bot
Copy link
Member Author

@MatrixAI-Bot
Copy link
Member Author

@CMCDragonkai
Copy link
Member

CMCDragonkai commented Jun 28, 2023

Ok so some problems with this right now:

  1. The pkg is at 5.8.1 but we need a release that uses pkg-fetch at 3.5, this is not yet released. So I suspect we need to wait for 5.8.2 or some later version for this to be doable. Might track this https://github.com/vercel/pkg-fetch/issues/285
  2. There's some problem with /homeless-shelter. This doesn't make sense to me as the HOME should exist already in the gitlab-runner. It seems to occur before the message Configured pkg with:. It doesn't really make sense in this context.

See:

> node ./scripts/pkg.js --no-dict=leveldown.js --output=out --bin=typescript-demo-lib --node-version=18 --platform=linux --arch=x64
npm ERR! code EACCES
npm ERR! syscall mkdir
npm ERR! path /homeless-shelter
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, mkdir '/homeless-shelter'
npm ERR!  [Error: EACCES: permission denied, mkdir '/homeless-shelter'] {
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'mkdir',
npm ERR!   path: '/homeless-shelter'
npm ERR! }
npm ERR! 
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR! 
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.
npm ERR! Log files were not written due to an error writing to the directory: /homeless-shelter/.npm/_logs
npm ERR! You can rerun the command with `--loglevel=verbose` to see the logs in your terminal
npm WARN config production Use `--omit=dev` instead.
Configured pkg with:

Something is trying to make /homeless-shelter.

The funny thing is that it says that something npm command tried to do this. But /homeless-shelter is sort of nix thing. So is it possible that HOME has been set to /homeless-shelter somehow? The only way to debug this is to echo $HOME.

We will leave this until pkg is ready to be updated and this will be relevant to Polykey too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants