forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commite2e34df
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 parent2ad35cf commite2e34df
2 files changed
+2
-2
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
271 | 271 |
| |
272 | 272 |
| |
273 | 273 |
| |
274 |
| - | |
| 274 | + | |
275 | 275 |
| |
276 | 276 |
| |
277 | 277 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
9261 | 9261 |
| |
9262 | 9262 |
| |
9263 | 9263 |
| |
9264 |
| - | |
| 9264 | + | |
9265 | 9265 |
| |
9266 | 9266 |
| |
9267 | 9267 |
| |
|
0 commit comments
Comments
(0)