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

Remove snapshot.blob from 18.x branch #46133

Closed
kapouer opened this issue Jan 8, 2023 · 8 comments
Closed

Remove snapshot.blob from 18.x branch #46133

kapouer opened this issue Jan 8, 2023 · 8 comments

Comments

@kapouer
Copy link
Contributor

kapouer commented Jan 8, 2023

Version

18.13.0

Platform

No response

Subsystem

No response

What steps will reproduce the bug?

No response

How often does it reproduce? Is there a required condition?

No response

What is the expected behavior?

Build artifacts shouldn't be in git repository, nor in the release tarball.

What do you see instead?

./snapshot.blob is in the git repository (branch 18.x) and in the release tarball of nodejs 18.13.0

Additional information

No response

@aduh95
Copy link
Contributor

aduh95 commented Jan 8, 2023

@danielleadams It looks like this was done when backporting 15f1051 on the v18.x branch in 2c0f7d4. It looks like something went wrong with the backport as the snapshot.blob was not there in the original commit.

@kapouer
Copy link
Contributor Author

kapouer commented Jan 8, 2023

I suppose it should go into .gitignore

@aduh95
Copy link
Contributor

aduh95 commented Jan 8, 2023

Do you want to send a PR?

@kapouer
Copy link
Contributor Author

kapouer commented Jan 8, 2023

Not really, other fish to fry

@ibrahimabba
Copy link

This is my first contribution to open source, i would love to work on this if this is something a beginner can do.
Thanks

@aduh95
Copy link
Contributor

aduh95 commented Jan 14, 2023 via email

@ibrahimabba
Copy link

Alright sir.

@juanarbol
Copy link
Member

Due to inactivity I'll proceed to fix this

juanarbol added a commit to juanarbol/node that referenced this issue Feb 12, 2023
The `snapshot.blob` file landed while backporting
nodejs@2c0f7d4
in the v18.x release line, but is not required

Fixes: nodejs#46133
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

No branches or pull requests

4 participants