Mmm, now that I think about it, if you do not have sphinx, everything succeeds, but the man page are actually not really man pages any more, but some sort of markdown or something. Please do not commit this.

In D41610#948401, @pjfloyd_wanadoo.fr wrote:

So basically being port maintainer means nothing here?

well, ok then :-)

I don't think it is a good idea, we've had a WWW line in the description for like 25 years, removing it would probably break tools, like freshports.

I can't seem to edit and propose a change for some reason, but, change it to something like this:

rebase

Could you maybe use temporary files (mktemp(1)) for fetch output instead of variables, it makes the jq lines shorter (jq '.foo' /tmp/file instead of echo $foo | jq ".foo")

Does this really have to be an extra patch? Does it break the build on other archs?

It feels like something like this should be in base, not in ports.

In D38634#879856, @fuz wrote:

The proposed commit message is now:

sysutils/rundeck3: update to 3.4.10
    
 - use options -H amd -o to daemon(8) for logging

Why was this change committed without a full exp-run?

poke @bapt

In D38523#877102, @eduardo wrote:

I understand what you say and no need to be frustrated about it.
This is a good to practice example about SED versus patches.

I cannot accept the same thing repeated over 10 file patches, I did not look at the rest of them: ~15 patches, without a better solution.

In D38429#874574, @vishwin wrote:

In D38429#874508, @mat wrote:

From what I understand, you removed sunpoet's fix because of some hypothetical future problem that may never happen.

This problem exists right now, because bytecode is not to be packaged, period. We (and many other operating system-level packagers) have been doing it wrong for years, and only as a crutch due to lack of a trigger mechanism that we now have. D34739 uses trigger support; having bytecode packaged hampers development of that, because proper operation cannot be verified.

In D38429#874574, @vishwin wrote:

Because you are running as root, which almost always owns the installed files.

From what I understand, you removed sunpoet's fix because of some hypothetical future problem that may never happen.

I have no idea what actually writes bytecode, or when it gets written, but it feels like this should be put in a place where it is picked up by all ports that will use some python bits, like directly in bsd.port.mk so all ports get it.

Two typos, and a simplification.