forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit6dce372
committed
Use --strip-unneeded when stripping static libraries with GNU strip.
We've long used "--strip-unneeded" for shared libraries but plain"-x" for static libraries when stripping symbols with GNU strip.There doesn't seem to be any really good reason for that though,since --strip-unneeded produces smaller output (as "-x" alonedoes not remove debug symbols). Moreover it seems thatllvm-strip, although it identifies as GNU strip, misbehaves whengiven "-x" for this purpose. It's unclear whether that'sintentional or a bug in llvm-strip, but in any case it seems likechanging to use --strip-unneeded in all cases should be a win.Note that this doesn't change our behavior when dealing withnon-GNU strip.Per gripes from Ed Maste and Palle Girgensohn. Back-patch,in case anyone wants to use llvm-strip with stable branches.Discussion:https://postgr.es/m/17898-5308d09543463266@postgresql.orgDiscussion:https://postgr.es/m/20230420153338.bbj2g5jiyy3afhjz@awork3.anarazel.de1 parent87d8ec3 commit6dce372
2 files changed
+2
-2
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
327 | 327 |
| |
328 | 328 |
| |
329 | 329 |
| |
330 |
| - | |
| 330 | + | |
331 | 331 |
| |
332 | 332 |
| |
333 | 333 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
9201 | 9201 |
| |
9202 | 9202 |
| |
9203 | 9203 |
| |
9204 |
| - | |
| 9204 | + | |
9205 | 9205 |
| |
9206 | 9206 |
| |
9207 | 9207 |
| |
|
0 commit comments
Comments
(0)