Skip to content

build: fix build error in sqlite under GN build #53686

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

zcbenz
Copy link
Contributor

@zcbenz zcbenz commented Jul 2, 2024

SQLite does not fix all compiler warnings so we have to disable the warnings on our side:
https://www.sqlite.org/faq.html#q17

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/security-wg

@zcbenz zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 2, 2024
@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Jul 2, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 3, 2024
@nodejs-github-bot

This comment was marked as outdated.

@zcbenz zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 4, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 4, 2024
@nodejs-github-bot

This comment was marked as outdated.

@zcbenz zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 5, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 5, 2024
@nodejs-github-bot
Copy link
Collaborator

@zcbenz
Copy link
Contributor Author

zcbenz commented Jul 5, 2024

The jenkins page shows Access Denied to me, is there something I can do to run "resume build"?

Screenshot 2024-07-05 at 11 24 26

@VoltrexKeyva
Copy link
Member

The jenkins page shows Access Denied to me, is there something I can do to run "resume build"?

Screenshot 2024-07-05 at 11 24 26

The CI is currently in lockdown, see thread.

@zcbenz zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 7, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 7, 2024
@nodejs-github-bot
Copy link
Collaborator

zcbenz added a commit that referenced this pull request Jul 7, 2024
PR-URL: #53686
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
@zcbenz
Copy link
Contributor Author

zcbenz commented Jul 7, 2024

Landed in 11f4efc

@zcbenz zcbenz closed this Jul 7, 2024
@zcbenz zcbenz deleted the sqlite-gn-build branch July 7, 2024 05:21
aduh95 pushed a commit that referenced this pull request Jul 12, 2024
PR-URL: #53686
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
@aduh95 aduh95 mentioned this pull request Jul 12, 2024
aduh95 pushed a commit that referenced this pull request Jul 16, 2024
PR-URL: #53686
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos targos added the dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. label Sep 21, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
dependencies Pull requests that update a dependency file. dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. needs-ci PRs that need a full CI run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants