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

Build sentry from source #706

Open
wants to merge 64 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
64 commits
Select commit Hold shift + click to select a range
e8cc79c
Unreal Build CMake
Dec 6, 2024
9ee4824
Link includes
Dec 6, 2024
f91ee10
Include platform paths
Dec 6, 2024
332f6ce
move native submodule
Dec 10, 2024
fcf3b7e
link library on windows
Dec 10, 2024
2ba8c1f
link Linux libraries
Dec 11, 2024
6422452
small build fixes
Dec 12, 2024
84a5cb6
load Xbox toolchain
Dec 16, 2024
3ed10ca
removed unused symlink
Dec 17, 2024
a94acbe
Fix submodules
Dec 17, 2024
81d74c5
Add changelog
Dec 17, 2024
c5bea5e
Fix xbox, add xbox one support and build backtrace option
PlasmaDev5 Dec 19, 2024
3b67fc9
Update package snapshot
tustanivsky Dec 19, 2024
a547a3a
Merge branch 'main' into feat/build-native
tustanivsky Dec 19, 2024
6efda5c
Update ci.yml
tustanivsky Dec 19, 2024
ad94624
build fix
PlasmaDev5 Dec 22, 2024
27760ef
Merge remote-tracking branch 'origin/feat/build-native' into feat/bui…
PlasmaDev5 Dec 22, 2024
14988c1
install cmake on build
PlasmaDev5 Dec 22, 2024
3ba3921
install cmake for tests
PlasmaDev5 Dec 22, 2024
6d6a100
Update CHANGELOG.md
PlasmaDev5 Dec 24, 2024
09aae1c
Review and CI fix
PlasmaDev5 Dec 24, 2024
c4fabe2
Merge remote-tracking branch 'origin/feat/build-native' into feat/bui…
Dec 24, 2024
15f266d
more fixes and improvements to cmake build
PlasmaDev5 Dec 24, 2024
823acdc
correct path to cmake
PlasmaDev5 Dec 24, 2024
e54ea8e
should be final paths
PlasmaDev5 Dec 27, 2024
99af459
Move include up to ensure its never missed
PlasmaDev5 Dec 27, 2024
7c45895
rename breakpad command to make more sense
PlasmaDev5 Dec 27, 2024
0a942a9
Add submodule checkout
tustanivsky Dec 30, 2024
389d3a1
Add ls
tustanivsky Dec 30, 2024
d54c0dd
Change case
tustanivsky Dec 30, 2024
ecf086d
Fix case in Build.cs
tustanivsky Dec 30, 2024
2e48327
Revert case change in ci script
tustanivsky Dec 30, 2024
b411c6e
link crashpad
PlasmaDev5 Jan 3, 2025
35bfe2e
Merge remote-tracking branch 'origin/feat/build-native' into feat/bui…
PlasmaDev5 Jan 3, 2025
438a344
build static
PlasmaDev5 Jan 3, 2025
ca91c90
fix static build
PlasmaDev5 Jan 3, 2025
24d1375
Build Fix
PlasmaDev5 Jan 7, 2025
6d457ea
Fix Linux paths
PlasmaDev5 Jan 7, 2025
fefa3fb
Copy Crashpad to file output
PlasmaDev5 Jan 7, 2025
7098d33
add missing dependencies for linux builds
Jan 7, 2025
5a9dc98
confine windows buildpath to windows
PlasmaDev5 Jan 7, 2025
7d14efe
fix checking for wrong file
PlasmaDev5 Jan 7, 2025
a45d5b6
re-order cmake command to maybe resolve linux issues
PlasmaDev5 Jan 7, 2025
35c08c0
make output dir if it doesnt exist
PlasmaDev5 Jan 8, 2025
4219c61
Get latest cmake
PlasmaDev5 Jan 8, 2025
0435848
linux CI fix
PlasmaDev5 Jan 8, 2025
b99e084
remove tree
PlasmaDev5 Jan 8, 2025
ac8c167
Try a diffrent way of getting cmake
PlasmaDev5 Jan 8, 2025
ceb6f9b
revert to base cmake isntall
PlasmaDev5 Jan 8, 2025
d9062bf
try diffrent approach for cmake
PlasmaDev5 Jan 8, 2025
bcbc87c
install cmake via pip
PlasmaDev5 Jan 8, 2025
84d3bae
install pip
PlasmaDev5 Jan 8, 2025
9da10e2
install correct pip
PlasmaDev5 Jan 8, 2025
30ebd43
small fix
PlasmaDev5 Jan 8, 2025
2568dc5
linux fixes
PlasmaDev5 Jan 8, 2025
c4cdfd3
trying to fix the EOF error
PlasmaDev5 Jan 8, 2025
1d49d68
fix EOF issues
PlasmaDev5 Jan 8, 2025
f4d902c
add bash
PlasmaDev5 Jan 8, 2025
b9c0d82
Fix crashpad handler build output path
tustanivsky Jan 9, 2025
8e088e9
log cmake path
PlasmaDev5 Jan 9, 2025
057b3ad
maybe need to echo the output so i can see it
PlasmaDev5 Jan 9, 2025
58cf28c
Try add cmake to path
PlasmaDev5 Jan 9, 2025
0dd083b
install wget
PlasmaDev5 Jan 9, 2025
ce3d067
Revert Linux
PlasmaDev5 Jan 27, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,8 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v4
with:
submodules: recursive

- name: Download CLI
shell: pwsh
Expand Down Expand Up @@ -224,6 +226,7 @@ jobs:
- uses: actions/checkout@v4
with:
path: checkout
submodules: recursive

- name: Extract package to ${{ matrix.app }}/Plugins
run: unzip sentry-unreal-*-engine${{ matrix.unreal }}-github.zip -d checkout/${{ matrix.app }}/Plugins/sentry
Expand All @@ -235,6 +238,8 @@ jobs:
- name: Run tests
id: run-tests
run: |
docker exec -w /workspace/checkout/${{ matrix.app }} unreal bash -c "
ls -al /workspace/checkout/${{ matrix.app }}/Plugins/sentry "
docker exec -w /workspace/checkout/${{ matrix.app }} unreal /home/ue4/UnrealEngine/Engine/Build/BatchFiles/RunUAT.sh BuildCookRun \
-project=/workspace/checkout/${{ matrix.app }}/SentryPlayground.uproject \
-archivedirectory=/workspace/checkout/${{ matrix.app }}/Builds \
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/sdk-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
if [[ "${{ inputs.target }}" == "Android" ]]; then
submodule="modules/sentry-java"
else
submodule="modules/sentry-native"
submodule="plugin-dev/sentry-native"
fi
echo "submodule=$submodule" >> $GITHUB_OUTPUT
echo "path=plugin-dev/Source/ThirdParty/${{ inputs.target }}" >> $GITHUB_OUTPUT
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/update-dependencies.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
native:
uses: getsentry/github-workflows/.github/workflows/updater.yml@v2
with:
path: modules/sentry-native
path: plugin-dev/sentry-native
name: Native SDK
secrets:
api-token: ${{ secrets.CI_DEPLOY_KEY }}
Expand Down
7 changes: 4 additions & 3 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
[submodule "modules/sentry-java"]
path = modules/sentry-java
url = https://github.com/getsentry/sentry-java.git
[submodule "modules/sentry-native"]
path = modules/sentry-native
url = https://github.com/getsentry/sentry-native.git

[submodule "plugin-dev/sentry-native"]
path = plugin-dev/sentry-native
url = https://github.com/getsentry/sentry-native.git
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

### Features

- Build the plugin through the Unreal Engine build system ([#706](https://github.com/getsentry/sentry-unreal/pull/706))

### Dependencies

- Bump Native SDK from v0.7.16 to v0.7.17 ([#717](https://github.com/getsentry/sentry-unreal/pull/717))
Expand Down
1 change: 0 additions & 1 deletion modules/sentry-native
Submodule sentry-native deleted from 407253
Loading
Loading