Forgot to note the review in the commit message, so manually closing the review.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Oct 4 2016
Sep 30 2016
Sep 29 2016
This review can be abandoned. The ipfilter support was added several months ago. Any problems with the code would need a new review.
Sep 28 2016
Sep 16 2016
With or without the comment change, the code change looks OK to me.
Sep 6 2016
Aug 30 2016
Aug 23 2016
This is the same as the last uploaded diff, only generated with -U99999 so the entire content of any affected files can be viewed through Phabricator.
Latest version of the diff, updated based on des' comments.
Aug 12 2016
Aug 5 2016
Jul 29 2016
Jul 15 2016
Refresh diff with latest code.
Updates based on comments received from des@ in email.
Jul 5 2016
Full diff uploaded, as requested by Ed Maste. Should otherwise be the same as the prior diff.
Jul 2 2016
I placed the option where you suggested in the Makefile, and
adjusted the order of the other parts of the Makefile to match.
Jun 23 2016
Jun 21 2016
I'm not opposed to this change. As I read the change, the default action is to still create EFI compliant GPT tables, with the minimum 128 partitions. I'm OK with the system growing the capability to create non-standards-conforming GPT tables, as long as the default is create conforming tables.
Jun 20 2016
I found my email exchange with Devin from earlier this year, and reworked the patch to follow his suggestions. The net effect is the same, a vendor supplied defaults files is sourced at the same point in the execution, just now named "/etc/defaults/vendor.conf" to match the other files named /etc/defaults/*.conf
The rc.local execution happens much later in the startup process, after daemons, etc have already started being executed.
Jun 19 2016
Jun 17 2016
Jun 12 2016
This review has been moved to be relative to the ports/sendmail infrastructure.
Jun 10 2016
Jun 9 2016
When you test this, make sure you test it with IPv6 traffic too.
Jun 8 2016
Updated to reflect first pass of review comments. Note - this round of changes hasn't been tested on my virtual machine (yet).
Jun 7 2016
Jun 6 2016
Jun 5 2016
Jun 4 2016
Jun 3 2016
Added NetBSD copyrights to appropriate files.
Rename 'a' variable to 'action' to better reflect its usage.
Jun 2 2016
Jun 1 2016
May 30 2016
Oops. This is incomplete. Doesn't handle the chroot() done for anonymous ftp properly. I will update with a more complete version soon.