forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commiteab2d31
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 parenta9781ae commiteab2d31
3 files changed
+3
-3
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
307 | 307 |
| |
308 | 308 |
| |
309 | 309 |
| |
310 |
| - | |
| 310 | + | |
311 | 311 |
| |
312 | 312 |
| |
313 | 313 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
9712 | 9712 |
| |
9713 | 9713 |
| |
9714 | 9714 |
| |
9715 |
| - | |
| 9715 | + | |
9716 | 9716 |
| |
9717 | 9717 |
| |
9718 | 9718 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
16 | 16 |
| |
17 | 17 |
| |
18 | 18 |
| |
19 |
| - | |
| 19 | + | |
20 | 20 |
| |
21 | 21 |
| |
22 | 22 |
| |
|
0 commit comments
Comments
(0)