Removed DOCSDIRS

Removed

In D48431#1105468, @0mp wrote:

One more question before we land it :)

I have no reasons to not approve this revision, but @garga as maintainer will have the last word

Updated commit message

In D48522#1107149, @0mp wrote:

Suggestions to the commit message:

www/apache24: Allow for overriding USERS and GROUPS via make.conf

It is a welcome addition for those building their own apache24 package
with custom UID/GID.

While here, use WWWOWN/WWWGRP instead of hardcoded www/www.

Also, please make sure to receive an approval from the maintainer before committing. Let me know when you get an approval from apache@freebsd.org.

In D48522#1107150, @0mp wrote:

BTW, please make sure that the www values are not hardcoded somewhere else as well:

1. Check the files/ directory.
2. Check the source code of apache for any obvious hardcoded user/group values for FreeBSD.

Added co-author and fixed author of the patch on the git commit

Used the port from the bugzilla and fixed few things.

Removed PORTREVISION used for my own poudriere

Added suggested changes

Done suggested changes !

PORTVERSION -> DISTVERSION

Updated pkg-descr for a better description, removed unused line in Makefile. Fixed the summary.

Use distutils instead of pep517

In D48147#1104408, @arrowd wrote:

I fail to see why the SSSD2 option is not removed completely. Flavors are dependable options. A flavor can be thought of as an option that other ports may depend on, because it produces a separate package.

With ports current state it is possible to get sudo-default with SSSD2 turned on and sudo-sssd with SSSD2 turned off.

Rebased patch, and fixed the summary since sssd2 is the only sssd option since Jan 1 2025.

In D48145#1103262, @bofh wrote:

Rebasing doesn't necessarily imply that you have to resubmit the patch. It's just a warning so that you don't make a mistake while committing depending on your workflow. Somehow I have a big list of tasks for a new comer when I mentor someone and I always do something to create a conflict so that they know how to fix it. I indeed missed the part of Approved by: as I normally don't bother about it too much; but yes make sure with others as it might make some people upset.

Added Approved by: bofh (maintainer) in the commit message as requested

This is updated diff with rebased port :)

Commited in https://cgit.freebsd.org/ports/commit/?id=2cf6bc1add649b35668b449ab35fb04ed0a6d263

Added - Pet portlint/portclippy as asked

Note I have swapped DISTVERSION and DISTVERSIONPREFIX because portclippy and portlint told me to do that.

Changed freebsd.org to FreeBSD.org as suggested.

Seems good to me.

Fix commit message

Variable names consitency.
Move do_resize=0 out of the if to avoid trying resize. May fail one time instead of always.

Fix logic

Do initialization of variables in main().
Renamed the fuction onsigwinch() into resizeit().
Check if return -1 before doing the next ioctl.

Added KASSERT() requested

Style fix for now

Fixed some style issues. On some files checkstyle9.pl is litterally yelling on
allmost all lines. Sorry to have tried to use same incorrect style of the file.
Added suggested #if defined on ipsec_output.c.

Better suggested way to avoid python bycode files than using rm.

Fixed some suggested changes.

• Fix some style bugs as requested (line too long / space before comment etc...)
• Make structure clearer on line 179 of udp6_usrreq.c
• Remove litteral
• Remove unecessary break

Typo on the filename

Configuration file should be in sample