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
F165170430: D46893.id144448.diff
Thu, Aug 6, 1:18 PM
F165151438: D46893.id144274.diff
Thu, Aug 6, 10:29 AM
Unknown Object (File)
Wed, Aug 5, 7:21 AM
Unknown Object (File)
Thu, Jul 30, 11:56 PM
Unknown Object (File)
Tue, Jul 28, 9:38 PM
Unknown Object (File)
Fri, Jul 24, 4:53 PM
Unknown Object (File)
Sun, Jul 19, 2:17 PM
Unknown Object (File)
Sat, Jul 18, 10:29 PM
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...