Page MenuHomeFreeBSD
Differential D31411

developers handbook: Remove obsolete information about binary objects
ClosedPublic
Actions

Authored by emaste on Aug 4 2021, 4:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 11, 2:07 PM
Unknown Object (File)
Mon, Nov 11, 1:53 PM
Unknown Object (File)
Mon, Nov 11, 1:49 PM
Unknown Object (File)
Mon, Nov 11, 10:23 AM
Unknown Object (File)
Mon, Nov 11, 10:18 AM
Unknown Object (File)
Mon, Nov 11, 10:18 AM
Unknown Object (File)
Mon, Nov 11, 10:17 AM
Unknown Object (File)
Mon, Nov 11, 10:17 AM
View All 46 Files
Subscribers
pauamma_gundo.com

Details

Reviewers
imp
ygy
Group Reviewers
docs
Commits
R9:41f56bf545e9: developers-handbook: remove obsolete object file commentary
Summary

It is no longer necessary to uuencode binary blobs for drivers. We can now commit the binary object to the tree as-is.

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste requested review of this revision.Aug 4 2021, 4:23 PM
emaste created this revision.
ygy accepted this revision.Aug 4 2021, 4:28 PM
This revision is now accepted and ready to land.Aug 4 2021, 4:28 PM
imp accepted this revision.Aug 4 2021, 4:33 PM

One suggested tweak you can take or leave. I'm fine with this or with my suggestion or with anything similar.

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

I'd add something like "Until converted to native format, " at the front of this rather than removing it entirely.

emaste added inline comments.Aug 4 2021, 4:39 PM
documentation/content/en/books/developers-handbook/policies/_index.adoc
274

Yeah, maybe something like "Historically, object files were uuencoded and named ..."

emaste updated this revision to Diff 93227.Aug 4 2021, 4:46 PM

Take @imp's suggestion, mention historical uuencoding.

This revision now requires review to proceed.Aug 4 2021, 4:46 PM
pauamma_gundo.com added a subscriber: pauamma_gundo.com.Aug 4 2021, 4:53 PM
pauamma_gundo.com added inline comments.
documentation/content/en/books/developers-handbook/policies/_index.adoc
274–275

Consensus is for a single sentence per line.

emaste updated this revision to Diff 93228.Aug 4 2021, 4:55 PM

One sentence per line (indented for readability as list item)

emaste marked 2 inline comments as done.Aug 4 2021, 4:56 PM
imp accepted this revision.Aug 4 2021, 5:22 PM
This revision is now accepted and ready to land.Aug 4 2021, 5:22 PM
Closed by commit R9:41f56bf545e9: developers-handbook: remove obsolete object file commentary (authored by emaste). · Explain WhyAug 4 2021, 6:29 PM
This revision was automatically updated to reflect the committed changes.
emaste added a commit: R9:41f56bf545e9: developers-handbook: remove obsolete object file commentary.

Revision Contents
Changeset List

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

Diff 93237

View Options

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

Loading...