Thanks for working on this, but I think pedantic compilation modes should be kept working.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Jun 10 2018
Jun 9 2018
Jun 7 2018
Jun 6 2018
May 30 2018
May 21 2018
I agree that the man page should be changed, but I think it can be done better.
May 20 2018
May 11 2018
May 6 2018
May 3 2018
Apr 29 2018
Apr 25 2018
Apr 18 2018
Apr 17 2018
The code itself looks good, but something could be improved in the man page and test.
Apr 15 2018
Even though it is from January, this is still a useful patch.
Apr 10 2018
I found two minor issues. It looks good otherwise.
Apr 8 2018
Apr 3 2018
Apr 2 2018
I like the idea. Apparently, the idea of the designers of setfacl was to pipe in a list of filenames from find, but this cannot handle filenames containing newlines and race conditions (such as where an untrusted user replaces a file with a symlink to a different location).
Thanks for writing missing documentation.
Mar 18 2018
In D14725#309715, @cem wrote:In D14725#309655, @jilles wrote:I think this kind of compatibility should be provided only if necessary for old code that cannot be changed (such as third party code). If it is just to avoid churn, we should keep using the old timespecadd and timespecsub (that is, abandon this change).
I don't follow that line of argument — that compatibility means we should abandon 3-arg timespecfoo, or bringing timespecfoo into userspace. But I will bow to quorum here, which seems to be: we'd rather churn it than have compatibility.
The changes to adapt to the new API look correct to me.
Mar 8 2018
This patch only affects the -d case. Failure to change owner and modes when installing regular files is fatal in any case. Perhaps the directory case should work the same way?
Mar 4 2018
This seems sensible, but I found some small mistakes in the man page.
Mar 2 2018
I agree that this should be based on unlinkat(), not unlink(). The new call will therefore take two file descriptors.
Feb 26 2018
Feb 18 2018
Feb 13 2018
I agree that hash table resizing would be nicer, but this is acceptable to commit.
Feb 6 2018
Feb 2 2018
Jan 24 2018
Hmm, it is remarkable to see this after the monotonic clock has existed for so long. Also, GNU time and mksh's keyword appear to use CLOCK_REALTIME without configurability.
Jan 21 2018
Jan 20 2018
This was committed by antoine as rS327473.
Jan 19 2018
It looks like it will work.
Jan 17 2018
getopts also allows option-arguments in the same argument as the option, like -jGAOL, or even -ejGAOL (for a jail named GAOL). Therefore, extracting the jail option out without rebuilding the arguments completely is rather hard.
Jan 14 2018
Jan 13 2018
Jan 11 2018
Jan 6 2018
Jan 5 2018
Jan 4 2018
It is already possible to disable swapping out kernel stacks via sysctl vm.swap_enabled (which has a wrong description) and configure it via sysctls vm.swap_idle_threshold1 and vm.swap_idle_threshold2. What is the problem you are trying to solve?
For a long time, "swapping out" a process has only made its U area (later, only the kernel stack) pageable. This is a few pages per thread. Therefore, comparing vmspace_resident_count does not seem to make much sense.
As of rS303524 (July 2016), _WITH_GETLINE is no longer checked by stdio.h. Therefore, there are two possibilities:
Jan 1 2018
Dec 31 2017
Dec 30 2017
Dec 29 2017
Looks useful.
Dec 27 2017
OK for bin/sh tests and man pages.
Dec 26 2017
Dec 24 2017
Dec 23 2017
Dec 10 2017
It looks like daemon()'s chdir and close actions are performed just before returning successfully; therefore, capability mode code could also call daemon(1, 1) and perform the chdir and/or close actions after it, either open-coded or calling new function(s).
Dec 5 2017
Sorry for the delay.
Dec 3 2017
Dec 2 2017
Nov 26 2017
This will leave the calls to /usr/bin/fortune in share/skel/dot.login and share/skel/dot.profile referring to something that does not and will not exist (the port will install as /usr/local/bin/fortune).
Nov 22 2017
Accepted with a minor change to the man page.
Nov 12 2017
Nov 11 2017
Nov 10 2017
Hmm, looking again, the idea of continuing is good, but printing diagnostics without affecting the exit status is usually not POSIX-compliant. To fix this, the global variable exitstatus can be set to 1. Note that there are existing places in function.c that do this wrong (which need not be fixed for this review).
Nov 3 2017
An alternative implementation would write to a temporary file, then rename onto the destination once stdin closes. This is how ksh93's >;pathname redirection works (this does not mean I think this definitely should be done like that, or that the shell is the best place to implement this feature).
Oct 31 2017
Note that all my comments on the cpuset_parselist() implementation also apply to the original version in cpuset.c; therefore, they could be fixed in a separate commit.
Oct 26 2017
Oct 22 2017
Oct 15 2017
Oct 6 2017
In D10363#260440, @theraven wrote:This probably means that we no longer need to expose _DefaultRuneLocale in the header.
Oct 1 2017
Looks good except for the missing DESTDIR error check.
Sep 26 2017
Sep 18 2017
As a side effect, this change brings in recent updates to the man page (this is good).
Aug 16 2017
In D12006#249341, @sevan wrote:@jilles best to just revert the commit & leave it as-is?