Page MenuHomeFreeBSD
Differential D46893

libkldelf: use warnx instead of printf
ClosedPublic
Actions

Authored by khng on Oct 4 2024, 1:11 AM.
Tags
None
Referenced Files
F154043914: D46893.id144274.diff
Sat, Apr 25, 5:04 PM
Unknown Object (File)
Mon, Apr 20, 10:38 AM
Unknown Object (File)
Mon, Apr 20, 8:25 AM
Unknown Object (File)
Mon, Apr 20, 5:49 AM
Unknown Object (File)
Mon, Apr 20, 2:30 AM
Unknown Object (File)
Sun, Apr 19, 11:50 PM
Unknown Object (File)
Sun, Apr 19, 11:10 PM
Unknown Object (File)
Sat, Apr 18, 2:02 AM
View All Files
Subscribers
imp

Details

Reviewers
markj
emaste
jhb
Commits
rG599c4399e9b8: libkldelf: use warnx instead of printf
rG550ee2d03c5b: libkldelf: use warnx instead of printf
Summary

Sponsored by: Juniper Networks, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 59764
Build 56650: arc lint + arc unit

Event Timeline

khng created this revision.Oct 4 2024, 1:11 AM
Herald added a subscriber: imp. · View Herald TranscriptOct 4 2024, 1:11 AM
khng requested review of this revision.Oct 4 2024, 1:11 AM
Harbormaster completed remote builds in B59702: Diff 144183.Oct 4 2024, 1:11 AM
khng added a parent revision: D46891: libkldelf: returns EOPNOTSUPP for unrecognizable relocations.Oct 4 2024, 1:11 AM
emaste accepted this revision.Oct 4 2024, 12:04 PM
This revision is now accepted and ready to land.Oct 4 2024, 12:04 PM
markj added a comment.Oct 4 2024, 12:33 PM

warnx() appends a newline, so this isn't going to give the same output.

emaste requested changes to this revision.Oct 4 2024, 12:37 PM
This revision now requires changes to proceed.Oct 4 2024, 12:37 PM
emaste added a comment.Oct 4 2024, 12:39 PM

Oh, yeah. The ones in ef_obj.c should be fine with the \ns removed.
ef_print_phdr is probably intended to go to stdout?

khng updated this revision to Diff 144274.Oct 4 2024, 9:23 PM

Fix per markj@'s comment. ef_print_phdr related part now is left unchanged for now.

Harbormaster completed remote builds in B59764: Diff 144274.Oct 4 2024, 9:23 PM
khng updated this revision to Diff 144275.Oct 4 2024, 9:24 PM

Oops.

Harbormaster completed remote builds in B59765: Diff 144275.Oct 4 2024, 9:24 PM
markj accepted this revision.Oct 4 2024, 11:43 PM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 8 2024, 4:29 AM
Closed by commit rG550ee2d03c5b: libkldelf: use warnx instead of printf (authored by khng). · Explain Why
This revision was automatically updated to reflect the committed changes.
khng added a commit: rG550ee2d03c5b: libkldelf: use warnx instead of printf.
khng added a reverting change: rG6df3a2d1bfed: Revert "libkldelf: use warnx instead of printf".Oct 8 2024, 7:48 PM
khng added a commit: rG599c4399e9b8: libkldelf: use warnx instead of printf.Oct 18 2024, 8:22 PM

Revision Contents
Changeset List

PathSize
lib/
libkldelf/
12 lines

Diff 144274

View Options

lib/libkldelf/ef_obj.c

Loading...