-
Notifications
You must be signed in to change notification settings - Fork 707
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
Brew formula #1275
Merged
Merged
Brew formula #1275
Changes from 19 commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
3a651ee
Feat: Creating a Homebrew Formula
dgreatwood e24a658
Feat: Enable GCC Builds on macOS and in macos.yaml Workflow
dgreatwood ea4dddd
Put bodySize direct in RB
dgreatwood 122f257
Fix: List Additional Headers in /include/pistache/meson.build
dgreatwood fb78dba
Fix: Improve Definition of PST_STR_ERROR_R Aross Platforms
dgreatwood f56366e
Feat: Updated release to dgreatwood/pistachefork v0.4.22
dgreatwood 70761fb
Fix: Revert "Put bodySize direct in RB"
dgreatwood a15c6ed
Fix: Address 4 Integer Type Issues seen on Arm32 Launchpad
dgreatwood c2f056c
Fix: Readme Updates
dgreatwood 6e45748
fix: avoid SIOF with lazy init
Finkman 53e60a6
fix: avoid macro replacements in includes
Finkman 3f50220
fix: ensure include pist_quote.h, remove duplicate includes
Finkman fe5a5a5
Update pist_strerror_r.cc
dgreatwood 8a505c0
Fix: Remove macOS 12 from GitHub Runner macos.yaml
dgreatwood ca9410b
Fix: Homebrew Formula Fixes
dgreatwood c22e2b6
Fix: Remove Duplicate basename_r + Deal with httplib.h Warnings
dgreatwood 0569035
Merge remote-tracking branch 'origin/master' into brewFormula
dgreatwood a1a002a
Fix: Don't Build Pistache Tests for Homebrew Formula
dgreatwood 9caa159
Merge remote-tracking branch 'origin/master' into brewFormula
dgreatwood a96e084
Fix: Reduce Unauthenticated GitHub Access to Avoid Rate Limit Issue
dgreatwood File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,91 @@ | ||
# SPDX-FileCopyrightText: 2022 Andrea Pappacoda | ||
# | ||
# SPDX-License-Identifier: Apache-2.0 | ||
|
||
# See: | ||
# https://docs.github.com/en/actions/writing-workflows/choosing-where-your-workflow-runs/choosing-the-runner-for-a-job | ||
# https://github.com/actions/runner-images/blob/main/images/macos/macos-14-Readme.md | ||
|
||
name: Homebrew Formula | ||
|
||
on: | ||
push: | ||
branches: | ||
- master | ||
pull_request: | ||
branches: | ||
- master | ||
|
||
defaults: | ||
run: | ||
shell: sh | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.ref }} | ||
cancel-in-progress: true | ||
|
||
jobs: | ||
macos: | ||
name: Homebrew Formula | ||
|
||
strategy: | ||
fail-fast: false | ||
matrix: | ||
os: [ 'macos-13', 'macos-14', 'macos-15' ] | ||
compiler: [ 'clang' ] | ||
sanitizer: [ 'none' ] | ||
tls: [ 'true' ] | ||
def_debug: [ 'false' ] | ||
|
||
runs-on: ${{ matrix.os }} | ||
|
||
steps: | ||
# Avoid doing "brew update" - the brew formulas that are | ||
# preinstalled on the github runner image are likely | ||
# consistent with the pre-installed software on the image. If | ||
# we do "brew upate", and then install something new with | ||
# brew, and the "something new" depends on pre-installed | ||
# software on the image, and there are new versions of the | ||
# pre-installed software revealed by doing "brew update", then | ||
# when we install the "something new" brew may try and also | ||
# install a new version of the pre-installed software on which | ||
# the "something new" depends, but that attempt to install a | ||
# new version of the pre-installed software can fail as a | ||
# result of being blocked by the software that is already | ||
# installed. | ||
|
||
- uses: actions/checkout@v4 | ||
with: | ||
submodules: true | ||
|
||
- name: Test Homebrew | ||
run: | | ||
if [ ${{ matrix.compiler }} = gcc ]; then | ||
gcc_lns=`brew list gcc | grep "bin/g++-[0-9][0-9]" | sort -r` | ||
gcc_fstln=`echo "$gcc_lns" | head -1` | ||
CXX=`basename $gcc_fstln` | ||
|
||
gcc_lns=`brew list gcc | grep "bin/gcc-[0-9][0-9]" | sort -r` | ||
gcc_fstln=`echo "$gcc_lns" | head -1` | ||
CC=`basename $gcc_fstln` | ||
else | ||
CXX=clang++ | ||
CC=CXX | ||
fi | ||
export CXX | ||
export CC | ||
echo "CXX is $CXX; CC is $CC" | ||
|
||
# Note - if you get the GitHub error: | ||
# GitHub API Error: API rate limit exceeded for aa.bb.cc.dd... | ||
# Then try rerunning the job in 20-25 minutes | ||
|
||
homebrew/runformula.sh -y --force --HEAD | ||
# -y Say "yes" to doing the brew audit | ||
# --force Run even if pistache brew formula unchanged | ||
# --HEAD Test with head of pistacheio/pistache master branch | ||
|
||
# if brew list pistache &>/dev/null; then brew remove pistache; fi | ||
# # Use release (not HEAD), do audit (-y), and force even if pistache | ||
# # formula unchanged | ||
# homebrew/runformula.sh -y --force |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -81,6 +81,10 @@ If you have no need to modify the Pistache source, you are strongly recommended | |
|
||
Pistache is available in the official repositories since Debian 12 and Ubuntu 23.10, under the package name `libpistache-dev`. | ||
|
||
### macOS | ||
|
||
Pistache can be installed using the Homebrew package manager. See *Building on macOS.txt* for specifics. | ||
|
||
#### Supported Architectures | ||
|
||
Currently Pistache is built and tested on a number of [architectures](https://wiki.debian.org/SupportedArchitectures). Some of these are suitable for desktop or server use and others for embedded environments. As of this writing we do not currently have any MIPS related packages that have been either built or tested. | ||
|
@@ -93,7 +97,7 @@ Currently Pistache is built and tested on a number of [architectures](https://wi | |
- riscv64 | ||
- s390x | ||
|
||
#### Ubuntu PPA (Unstable) | ||
### Ubuntu PPA (Unstable) | ||
|
||
The project builds [daily unstable snapshots](https://launchpad.net/~pistache+team/+archive/ubuntu/unstable) in a separate unstable PPA. To use it, run the following: | ||
|
||
|
@@ -103,7 +107,7 @@ $ sudo apt update | |
$ sudo apt install libpistache-dev | ||
``` | ||
|
||
#### Ubuntu PPA (Stable) | ||
### Ubuntu PPA (Stable) | ||
|
||
Currently there are no stable release of Pistache published into the [stable](https://launchpad.net/~pistache+team/+archive/ubuntu/stable) PPA. However, when that time comes, run the following to install a stable package: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @dgreatwood, this isn't really an issue you created, but feel free to fix this paragraph to reflect the fact that we now have stable releases in the stable PPA. |
||
|
||
|
@@ -205,8 +209,7 @@ To download the latest available release, clone the repository over GitHub. | |
$ git clone https://github.com/pistacheio/pistache.git | ||
``` | ||
|
||
To build for macOS, you can follow the instructions in: | ||
*Building on macOS.txt* | ||
To build on macOS, Windows, or BSD, see the respective files *Building on macOS.txt*, *Building on Windows.txt* or *Building on BSD.txt*. | ||
|
||
Continuing the Linux instructions: | ||
|
||
|
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't write this, I do not own copyright on the file :)