Skip to content

[llhttp] 9.4.2-1: init package - #7976

Open
WeirdGumi wants to merge 1 commit into
eweOS:llhttpfrom
WeirdGumi:llhttp
Open

[llhttp] 9.4.2-1: init package#7976
WeirdGumi wants to merge 1 commit into
eweOS:llhttpfrom
WeirdGumi:llhttp

Conversation

@WeirdGumi

Copy link
Copy Markdown

No description provided.

Comment thread PKGBUILD

prepare() {
cd $pkgname-$pkgver
npm install

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is quite uncommon, why we must run npm install first? Please comment on this in PKGBUILD.

Comment thread PKGBUILD
Comment on lines +18 to +23
make RELEASE=$pkgver release
}

build() {
cmake -S $pkgname-$pkgver/release -B build -DCMAKE_INSTALL_PREFIX=/usr
cmake --build build

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's quite uncommon, too, that both bare make and cmake are used. Please explain this in PKGBUILD.

@github-project-automation github-project-automation Bot moved this from New to ReqChanges in eweOS Package Management Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ReqChanges

Development

Successfully merging this pull request may close these issues.

2 participants