Page MenuHomeFreeBSD
Differential D31213

developers-handbook: update shared libraries policy section
ClosedPublic
Actions

Authored by kib on Jul 18 2021, 1:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 14 2024, 10:51 PM
Unknown Object (File)
Jan 11 2024, 1:37 AM
Unknown Object (File)
Dec 16 2023, 2:18 AM
Unknown Object (File)
Nov 10 2023, 10:45 PM
Unknown Object (File)
Nov 8 2023, 10:51 PM
Unknown Object (File)
Nov 8 2023, 7:31 PM
Unknown Object (File)
Nov 4 2023, 11:00 PM
Unknown Object (File)
Oct 29 2023, 5:53 AM
View All 28 Files
Subscribers
pauamma_gundo.com

Details

Reviewers
emaste
Group Reviewers
docs
Commits
R9:aaa7af03ec91: developers-handbook: update shared libraries policy section
Summary

Current text was written with a.out shared libraries system in mind, which is not adequate for ELF system. In fact it can be considered as mis-information causing to spread false rumors about FreeBSD environment.

Drop everything that comes from a.out world, and briefly describe common conventions related to ELF shared objects.

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib requested review of this revision.Jul 18 2021, 1:54 PM
kib created this revision.
pauamma_gundo.com added a subscriber: pauamma_gundo.com.Jul 18 2021, 3:06 PM

Note: I don't know what the current policy and practice are, so I can't comment on how accurately the change reflects them.

documentation/content/en/books/developers-handbook/policies/_index.adoc
292–293
295
301

I'd change "argumentation" to either "arguments" or "reasons"

305
307–311

Also, 1 sentence per line? I think doc people settled on that.

kib updated this revision to Diff 92398.Jul 18 2021, 7:13 PM
kib marked 5 inline comments as done.

Some grammar.

pauamma_gundo.com added inline comments.Jul 18 2021, 9:09 PM
documentation/content/en/books/developers-handbook/policies/_index.adoc
308
309
310
311
kib updated this revision to Diff 92401.Jul 18 2021, 10:26 PM
kib marked 4 inline comments as done.

More grammar.

emaste accepted this revision.Jul 27 2021, 7:11 PM

lgtm, a couple of minor items that you could consider

documentation/content/en/books/developers-handbook/policies/_index.adoc
300

I would write

  • If this is impossible, or the library does not use symbol versioning, bump the library version
301

perhaps spell out Release Engineering, lowercase re looks like a fragment of a missing word

309

I might use libexample.so.x.y rather than libmy.so.x.y

This revision is now accepted and ready to land.Jul 27 2021, 7:11 PM
kib marked 3 inline comments as done.Jul 27 2021, 7:47 PM
Closed by commit R9:aaa7af03ec91: developers-handbook: update shared libraries policy section (authored by kib). · Explain WhyJul 27 2021, 7:49 PM
This revision was automatically updated to reflect the committed changes.
kib added a commit: R9:aaa7af03ec91: developers-handbook: update shared libraries policy section.

Revision Contents
Changeset List

PathSize
documentation/
content/
en/
books/
developers-handbook/
policies/
37 lines

Diff 92401

View Options

documentation/content/en/books/developers-handbook/policies/_index.adoc

Loading...