-
-
Notifications
You must be signed in to change notification settings - Fork 384
Creation of pgrouting version 4.0.0-alpha1 #2970
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
Changes from all commits
44af515
069880b
588122a
86aa247
0273e06
754be73
d4d7df3
6c15518
5eeedd5
79c6582
d6b7fab
4ee3462
9404143
e70aead
b14469c
4885035
c3e092e
03fe386
30d45a2
15d04f1
de39f95
b0821fe
43cb2d9
0bc0fd0
f582cfe
bd2cff4
8079ecf
cd2bdad
402a4fb
8279c69
eada7c9
3841c51
898aac8
3de77e0
d6c40f6
0e7f8c0
6ecd28e
e88aac3
e33c7b2
77def58
40d3456
1dfe32c
2d26bbb
03432af
b12f27f
bcc8285
7804a09
3d91528
47c2691
de8a2cb
b9c5d6f
6219579
e84518a
d3e6c67
fdbaa3f
0f8746d
5cde171
25a66b8
9c4fd71
18d1450
caa44c2
10b133b
1067688
a560220
934be38
0d8f64e
830da0b
60e6f39
716163e
97de361
9232fd8
955eb7b
fffdd39
2b5ebac
468caf7
071ee97
2a3d696
6797aad
b39933f
c44a4b8
6a3dfc1
fe11d28
35d6695
1bedc89
a212306
c9473f6
996c1e2
e4657c6
0cbd9c8
f4b6190
bc1ee8f
013a150
f68eab2
7ae558b
a18d2f7
ee7ff37
0fbbd1e
fdab289
43c228b
5b8f2a7
4ff7e96
6575e64
a045af5
e66bd50
70ea35a
d143855
4be9ddd
f071631
b9b2b20
8fa118f
bafcf64
27cb0fb
452ab96
2dee6b4
f565636
2b7450c
33c0226
6e774b0
f8350e2
4062932
13e003a
191a2d6
b8349a2
2d417e6
2eee998
9788e68
732153d
a41dd78
4b7ba04
5a6a0f1
8a402f9
5debd61
715c4f7
35562ef
76abf22
c3818c8
cbe1dfc
4271b17
a161f38
94cc857
602a324
4bca9cc
80c7615
cfe6384
6599bd6
e87148f
9df2bb3
01d9e1f
a0c7281
0446d44
dee03ee
ef46950
81c5b18
bf2324d
ddf5a69
56ba85a
450b7a6
45c588c
de15dde
8fd514f
95754ad
150d3f1
1484c50
be70aac
4edc4bc
4275f2c
50ac5fa
83a3201
b36839d
1143c80
3285b24
87f94df
e3097cd
cfc8f92
add26fc
c8f4646
2ed2c6d
9eadf7a
e19f387
7c9193b
f65a300
f1e6478
b5ec2bb
6532eb9
1894d12
e822898
80d70b4
f08b768
81204c8
facfa73
6edbe0c
ab32ee4
375976f
257c53d
622af61
fda3fef
6f4397e
fc5665e
ba8ab51
6a52ca7
3012128
a7155f2
c893dd1
30341df
2e801bd
c387f9a
4cd6337
15d1f75
7500775
4b30a59
148e09b
14297e4
95c0110
cf0f6ca
ed67f20
907d643
241adca
db47453
c89ff36
6cbe34a
a92d47c
e6890b9
63e8227
7ed9fb1
398295f
327cd96
b80e544
7edf506
8e5f934
43ddaa8
b0017f3
ebb83ad
912620b
eb8934b
65a8b41
229f1b7
da2ec0c
d95f085
7a3086c
4a6e8a7
06dfa99
61a3012
901f608
414d1ee
efd9d8c
ebf8876
51c219e
91d6e20
2219bdb
105513c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| Checks: > | ||
| google-*, | ||
| -google-readability-casting, | ||
| clang-analyzer-*, | ||
| clang-diagnostic-*, | ||
| cppcoreguidelines-avoid-capturing-lambda-coroutines, | ||
| cppcoreguidelines-avoid-goto, | ||
| cppcoreguidelines-avoid-non-const-global-variables, | ||
| cppcoreguidelines-avoid-reference-coroutine-parameters | ||
|
|
||
| WarningsAsErrors: '' | ||
| HeaderFilterRegex: './include' | ||
| FormatStyle: none | ||
| InheritParentConfig: true |
This file was deleted.
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -19,21 +19,19 @@ on: | |||||||||||||||||||||||||||||||||||||
| - 'include/**' | ||||||||||||||||||||||||||||||||||||||
| - 'pgtap/**' | ||||||||||||||||||||||||||||||||||||||
| - 'tools/testers/**' | ||||||||||||||||||||||||||||||||||||||
| - './CMakeLists.txt' | ||||||||||||||||||||||||||||||||||||||
| - 'CMakeLists.txt' | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| branches-ignore: | ||||||||||||||||||||||||||||||||||||||
| - 'gh-pages' | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| tags: [] | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| pull_request: | ||||||||||||||||||||||||||||||||||||||
| paths: | ||||||||||||||||||||||||||||||||||||||
| - '.github/workflows/clang.yml' | ||||||||||||||||||||||||||||||||||||||
| - 'src/**' | ||||||||||||||||||||||||||||||||||||||
| - 'include/**' | ||||||||||||||||||||||||||||||||||||||
| - 'pgtap/**' | ||||||||||||||||||||||||||||||||||||||
| - 'tools/testers/**' | ||||||||||||||||||||||||||||||||||||||
| - './CMakeLists.txt' | ||||||||||||||||||||||||||||||||||||||
| - 'CMakeLists.txt' | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| branches-ignore: | ||||||||||||||||||||||||||||||||||||||
| - 'gh-pages' | ||||||||||||||||||||||||||||||||||||||
|
|
@@ -48,10 +46,13 @@ permissions: | |||||||||||||||||||||||||||||||||||||
| jobs: | ||||||||||||||||||||||||||||||||||||||
| Test_clang: | ||||||||||||||||||||||||||||||||||||||
| name: Ubuntu clang | ||||||||||||||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||||||||||||||
| runs-on: ${{ matrix.os }} | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| strategy: | ||||||||||||||||||||||||||||||||||||||
| fail-fast: false | ||||||||||||||||||||||||||||||||||||||
| matrix: | ||||||||||||||||||||||||||||||||||||||
| release: [Debug, Release] | ||||||||||||||||||||||||||||||||||||||
| os: [ubuntu-latest, ubuntu-22.04] | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
coderabbitai[bot] marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||||||||
| - uses: actions/checkout@v5 | ||||||||||||||||||||||||||||||||||||||
cvvergara marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||
|
|
@@ -64,12 +65,21 @@ jobs: | |||||||||||||||||||||||||||||||||||||
| echo "PGIS=3" >> $GITHUB_ENV | ||||||||||||||||||||||||||||||||||||||
| echo "PGPORT=5432" >> $GITHUB_ENV | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| - name: 'Raise Priority for apt.postgresql.org' | ||||||||||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||||||||||
| cat << EOF >> ./pgdg.pref | ||||||||||||||||||||||||||||||||||||||
| Package: * | ||||||||||||||||||||||||||||||||||||||
| Pin: release o=apt.postgresql.org | ||||||||||||||||||||||||||||||||||||||
| Pin-Priority: 600 | ||||||||||||||||||||||||||||||||||||||
| EOF | ||||||||||||||||||||||||||||||||||||||
| sudo mv ./pgdg.pref /etc/apt/preferences.d/ | ||||||||||||||||||||||||||||||||||||||
| sudo apt update | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+68
to
+77
Contributor
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. 🧹 Nitpick (assertive) APT pinning step: OK, but keep file mode and idempotency in mind. Consider using install -m 644 to set perms explicitly; current approach is acceptable. - cat << EOF >> ./pgdg.pref
+ install -m 644 /dev/stdin ./pgdg.pref << 'EOF'📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||
| - name: Add PostgreSQL APT repository | ||||||||||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||||||||||
| sudo apt-get install curl ca-certificates gnupg | ||||||||||||||||||||||||||||||||||||||
| curl https://www.postgresql.org/media/keys/ACCC4CF8.asc | sudo apt-key add - | ||||||||||||||||||||||||||||||||||||||
| sudo sh -c 'echo "deb http://apt.postgresql.org/pub/repos/apt/ \ | ||||||||||||||||||||||||||||||||||||||
| $(lsb_release -cs)-pgdg main" > /etc/apt/sources.list.d/pgdg.list' | ||||||||||||||||||||||||||||||||||||||
| sudo apt-get -y purge postgresql-* | ||||||||||||||||||||||||||||||||||||||
| sudo sh -c 'echo "deb http://apt.postgresql.org/pub/repos/apt $(lsb_release -cs)-pgdg-testing main ${PGVER}" > /etc/apt/sources.list.d/pgdg.list' | ||||||||||||||||||||||||||||||||||||||
| curl https://www.postgresql.org/media/keys/ACCC4CF8.asc | gpg --dearmor | sudo tee /etc/apt/trusted.gpg.d/apt.postgresql.org.gpg >/dev/null | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| - name: Install dependencies | ||||||||||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||||||||||
|
|
@@ -84,12 +94,14 @@ jobs: | |||||||||||||||||||||||||||||||||||||
| postgresql-${PGVER}-postgis-${PGIS}-scripts \ | ||||||||||||||||||||||||||||||||||||||
| postgresql-server-dev-${PGVER} | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| - name: Configure | ||||||||||||||||||||||||||||||||||||||
| - name: Configure for clang | ||||||||||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||||||||||
| export PATH=/usr/lib/postgresql/${PGVER}/bin:$PATH | ||||||||||||||||||||||||||||||||||||||
| mkdir build | ||||||||||||||||||||||||||||||||||||||
| cd build | ||||||||||||||||||||||||||||||||||||||
| CXX=clang++ CC=clang cmake -DPOSTGRESQL_VERSION=${PGVER} -DCMAKE_BUILD_TYPE=Debug -DWITH_DOC=OFF .. | ||||||||||||||||||||||||||||||||||||||
| CXX=clang++ CC=clang cmake -DPOSTGRESQL_VERSION=${PGVER} \ | ||||||||||||||||||||||||||||||||||||||
| -DCMAKE_BUILD_TYPE=${{ matrix.release }} -DBUILD_HTML=OFF -DBUILD_DOXY=OFF \ | ||||||||||||||||||||||||||||||||||||||
| -DUSE_CLANG_TIDY=ON .. | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+97
to
105
Contributor
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. 🧹 Nitpick (assertive) Good: clang-tidy enabled and HTML/DOXY disabled for speed. Also pass -DBUILD_CCOV=OFF if present to trim deps. 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||
| - name: Build | ||||||||||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
Uh oh!
There was an error while loading. Please reload this page.