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
F171041194: D31411.id93228.diff
Tue, Sep 8, 8:41 AM
Unknown Object (File)
Mon, Sep 7, 1:12 PM
Unknown Object (File)
Mon, Sep 7, 1:09 PM
Unknown Object (File)
Mon, Sep 7, 1:00 PM
Unknown Object (File)
Mon, Sep 7, 12:52 PM
Unknown Object (File)
Mon, Sep 7, 12:51 PM
Unknown Object (File)
Wed, Sep 2, 11:19 AM
Unknown Object (File)
Mon, Aug 31, 8:04 AM
View All Files
Subscribers
pauamma_chezdork.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

Lint
Lint Skipped
Unit
Tests Skipped

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_chezdork.com added a subscriber: pauamma_chezdork.com.Aug 4 2021, 4:53 PM
pauamma_chezdork.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 93228

View Options

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

Loading...