forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commita14afd3
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 parent63a03ae commita14afd3
2 files changed
+2
-2
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
309 | 309 |
| |
310 | 310 |
| |
311 | 311 |
| |
312 |
| - | |
| 312 | + | |
313 | 313 |
| |
314 | 314 |
| |
315 | 315 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
9669 | 9669 |
| |
9670 | 9670 |
| |
9671 | 9671 |
| |
9672 |
| - | |
| 9672 | + | |
9673 | 9673 |
| |
9674 | 9674 |
| |
9675 | 9675 |
| |
|
0 commit comments
Comments
(0)