Skip to content

apache.org/httpd & apr-util --includedir fix#1798

Merged
jhheider merged 8 commits into
pkgxdev:mainfrom
and-ri:apache.org/httpd
May 1, 2023
Merged

apache.org/httpd & apr-util --includedir fix#1798
jhheider merged 8 commits into
pkgxdev:mainfrom
and-ri:apache.org/httpd

Conversation

@and-ri
Copy link
Copy Markdown
Contributor

@and-ri and-ri commented May 1, 2023

closed #333

@what-the-diff
Copy link
Copy Markdown

what-the-diff Bot commented May 1, 2023

PR Summary

  • Added a new package: httpd
    The pull request includes the addition of a new package, httpd, which enhances the project's capabilities.

  • Improved apr-util package build script
    The apr-util package's build script has been updated with a new parameter --includedir={{prefix}}/include, which ensures better organization and management of the package on the system.

@and-ri
Copy link
Copy Markdown
Contributor Author

and-ri commented May 1, 2023

Hm strange 🧐 Everything worked on my mac.

@jhheider
Copy link
Copy Markdown
Contributor

jhheider commented May 1, 2023

that fact that's it's trying to make directories that already exist suggest that the concurrency could be at issue. try make without -j.

@and-ri
Copy link
Copy Markdown
Contributor Author

and-ri commented May 1, 2023

Something wrong

building apache.org/apr-util>=1.3
error: Uncaught (in promise) Error: not-array: undefined
  if (!isArray(input)) throw new Error(`not-array: ${JSON.stringify(input)}`)
                             ^
    at validate_arr (https://github.com/teaxyz/cli/v0.30/src/utils/index.ts:21:30)
    at getVersionsLong (file:///opt/tea.xyz/brewkit/v0.26.0/lib/useGitHubAPI.ts:113:19)
    at eventLoopTick (ext:core/01_core.js:166:11)
    at async Object.getVersions (file:///opt/tea.xyz/brewkit/v0.26.0/lib/useGitHubAPI.ts:40:[29](https://github.com/teaxyz/pantry/actions/runs/4855605843/jobs/8654327461?pr=1798#step:7:30))
    at async handleAPIResponse (file:///opt/tea.xyz/brewkit/v0.26.0/lib/usePantry.ts:412:20)
    at async Object.resolve (file:///opt/tea.xyz/brewkit/v0.26.0/lib/usePantry.ts:63:20)
    at async file:///opt/tea.xyz/brewkit/v0.26.0/libexec/resolve.ts:18:15
Error: Process completed with exit code 1.

@jhheider
Copy link
Copy Markdown
Contributor

jhheider commented May 1, 2023

That's similar to what I was getting for vim.org: pkgxdev/brewkit#103

Makes me wonder if GitHub's GraphQL endpoint it ailing; except that it works locally. GHA might be more susceptible to a slow endpoint (or it might be getting different responses than we).

@jhheider
Copy link
Copy Markdown
Contributor

jhheider commented May 1, 2023

That said, the fix in that PR fixes vim.org

@and-ri
Copy link
Copy Markdown
Contributor Author

and-ri commented May 1, 2023

that fact that's it's trying to make directories that already exist suggest that the concurrency could be at issue. try make without -j.

Wow, it worked. 🥳 Thank you, I would spend a lot of time to find this solution

@jhheider
Copy link
Copy Markdown
Contributor

jhheider commented May 1, 2023

That basically means that the makefile isn't set up with all the correct dependencies, since it's can parallelize something that needs to be concurrent.

@jhheider jhheider merged commit e5c9014 into pkgxdev:main May 1, 2023
@and-ri and-ri deleted the apache.org/httpd branch May 2, 2023 08:06
@jhheider jhheider mentioned this pull request May 3, 2023
@magnusviri
Copy link
Copy Markdown
Contributor

I was curious why apachectl wasn't listed in the provides section. I use apachectl on all of my web servers.

I also know it's easy to get around this error by specifying the config on the cli, but by default running httpd produces this error.

httpd: Could not open configuration file /opt/apache.org/httpd/v2.4.57/etc/httpd/httpd.conf: No such file or directory

@and-ri
Copy link
Copy Markdown
Contributor Author

and-ri commented May 4, 2023

I was curious why apachectl wasn't listed in the provides section. I use apachectl on all of my web servers.

I also know it's easy to get around this error by specifying the config on the cli, but by default running httpd produces this error.

httpd: Could not open configuration file /opt/apache.org/httpd/v2.4.57/etc/httpd/httpd.conf: No such file or directory

Yes indeed you are right, thanks. I will fix 🤓

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

Successfully merging this pull request may close these issues.

+httpd (195/548)

3 participants