{"push_id":1765530699,"size":2,"distinct_size":2,"ref":"refs/heads/stable/10","head":"ee12e5f22f8bea038bd3c0fe652d39229c92b4a6","before":"75e7ac62b1cfd9030af1e58bf8ac1fa63c888c28","commits":[{"sha":"f66e567e6e91901615fef15cd822054816aee044","author":{"name":"ngie","email":"305547d4cc311567c72a2740c290593f0de39607@FreeBSD.org"},"message":"MFC r316099:\n\nlib/libkvm: start adding basic tests for kvm(3)\n\n- kvm_close: add a testcase to verify support for errno = EINVAL / -1\n  (see D10065) when kd == NULL is provided to the libcall.\n- kvm_geterr:\n-- Add a negative testcase for kd == NULL returning \"\" (see D10022).\n-- Add two positive testcases:\n--- test the error case using kvm_write on a O_RDONLY descriptor.\n--- test the \"no error\" case using kvm_read(3) and kvm_nlist(3) as\n    helper routines and by injecting a bogus error message via\n    _kvm_err (an internal API) _kvm_err was used as there isn't a\n    formalized way to clear the error output, and because\n    kvm_nlist always returns ENOENT with the NULL terminator today.\n- kvm_open, kvm_open2:\n-- Add some basic negative tests for kvm_open(3) and kvm_open2(3).\n   Testing positive cases with a specific\n   `corefile`/`execfile`/`resolver` requires more work and would require\n   user intervention today in order to reliably test this out.\n\nMFC note:\n  lib/libkvm/kvm_open2_test is not compiled/tested because ^/stable/10\n  lacks the kvm_open2(3) libcall.","distinct":true,"url":"https://api.github.com/repos/freebsd/freebsd/commits/f66e567e6e91901615fef15cd822054816aee044"},{"sha":"ee12e5f22f8bea038bd3c0fe652d39229c92b4a6","author":{"name":"ngie","email":"305547d4cc311567c72a2740c290593f0de39607@FreeBSD.org"},"message":"MFC r317290,r317291,r317292,r317293,r317294,r317295,r317304,r317306,r317307,r317308,r317309:\n\nr317290:\n\nFix -Wimplicit-function-declaration compilation warning by moving libgeom.h\n#include below the stdio.h #include.\n\ngctl_dump(3) needs stdio.h, per reasoning noted in r317289.\n\nPR:\t\t218809\n\nr317291:\n\nRename gctl.t to gctl_test.t and test.c to gctl_test_helper.c\n\nThis is being done to reduce ambiguity and to make the tests more portable\nin the future to other locations in the source tree.\n\nr317292:\n\ngctl_test.t: use make to compile gctl_test_helper instead of calling cc directly\n\nr317293:\n\ngctl_test_helper: apply polish\n\n- Staticize variables to fix warnings.\n- Sprinkle asserts around for calls that can fail\n- Apply style(9) for main(..) definition.\n- ANSIify usage(..) definition.\n\nr317294:\n\nBump WARNS to 6 per previous commits which fixed warnings\n\nTested with:\tclang (4.0), gcc (4.2.1, 6.3.0)\n\nr317295:\n\nThe GPT class no longer exists; use the PART class instead\n\nr317304:\n\ngctl_test_helper: add diagnostic output for parse_retval(..)\n\nThis will help end-users better diagnose issues with the function.\n\nr317306:\n\ngctl_test.t: minor tweaks\n\n- Declare $count with the `my` scope operator to permit `use strict`.\n- Add `use strict`.\n- Use `use warnings` instead of using `-w` in the shebang.\n- Don't unlink $cmd when done (prevents unnecessary rebuilding).\n- Improve the error message when running with insufficient permissions, e.g.,\n  non-root.\n\nr317307:\n\nUse verb=delete not verb=remove\n\nThe `remove` verb hasn't been present in geom_part*(4) for well\nover a decade, if ever. I couldn't find any references to it in\n^/stable/5 at least, which is around the timeframe that this test\nwas written.\n\nr317308:\n\ngctl_test.t: more tweaks to try and update the code and get it functional (again?)\n\n- Make the logfile for $out be built off the basename for $cmd, instead of $cmd.\n  (r317292 broke this assumption).\n- Rename $mntpt to $mntpt_prefix for clarity, as this variable is a prefix for\n  mountpoints.\n- Reindent the umount directive block while here to match the rest of the code.\n\nr317309:\n\ngctl_test.t: improve error reporting with mdcfg and mount directives\n\nIf the commands had failed previously, it would press on and result in a\nseries of cascading failures. Fail early and continue on to the next case\ninstead of executing additional commands after a previously failed series\nof steps.","distinct":true,"url":"https://api.github.com/repos/freebsd/freebsd/commits/ee12e5f22f8bea038bd3c0fe652d39229c92b4a6"}]}
{"actor":{"display_login":"uqs"}}