Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

syslogd: fix WITHOUT_INET builds #475

Closed

Conversation

akhramov
Copy link
Contributor

Since 2d82b47 syslogd can't be built with WITHOUT_INET or
WITHOUT_INET6 build variables set, because iovlist_truncate is not
defined but used.

This change wraps the problematic iovlist_truncate call within ifdef
directive.

Since 2d82b47 syslogd can't be built with `WITHOUT_INET` or
`WITHOUT_INET6` build variables set, because `iovlist_truncate` is not
defined but used.

This change wraps the problematic `iovlist_truncate` call within ifdef
directive.
@bsdimp
Copy link
Member

bsdimp commented Jun 13, 2021

This is correct. I did a minor tweak to the commit message to explain why this is ok better. iovlist_truncate is compiled out for this situation, and should have been noted in the commit message. Thanks!
f06fa6f

@bsdimp bsdimp closed this Jun 13, 2021
freebsd-git pushed a commit that referenced this pull request Jun 13, 2021
Since 2d82b47 syslogd can't be built with `WITHOUT_INET` or
`WITHOUT_INET6` build variables set, because `iovlist_truncate` is not
defined but used.

This change wraps the problematic `iovlist_truncate` call within ifdef
directive.  It's compiled out in this situation...

Pull Request:	#475
Reviewed by:	imp@ (commit message slightly tweaked)
@bsdimp
Copy link
Member

bsdimp commented Jun 13, 2021

Commit f06fa6f is what's needed.

rozhuk-im pushed a commit to rozhuk-im/freebsd that referenced this pull request Nov 29, 2021
Since 2d82b47 syslogd can't be built with `WITHOUT_INET` or
`WITHOUT_INET6` build variables set, because `iovlist_truncate` is not
defined but used.

This change wraps the problematic `iovlist_truncate` call within ifdef
directive.  It's compiled out in this situation...

Pull Request:	freebsd#475
Reviewed by:	imp@ (commit message slightly tweaked)

(cherry picked from commit f06fa6f)

PR:		260098
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Dec 11, 2021
Since 2d82b47 syslogd can't be built with `WITHOUT_INET` or
`WITHOUT_INET6` build variables set, because `iovlist_truncate` is not
defined but used.

This change wraps the problematic `iovlist_truncate` call within ifdef
directive.  It's compiled out in this situation...

Pull Request:	freebsd/freebsd-src#475
Reviewed by:	imp@ (commit message slightly tweaked)
@emaste emaste added the merged label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants