Diederik has submitted this change and it was merged. Change subject: Added switch for dump directory to collector ......................................................................
Added switch for dump directory to collector 1) Removed whitespace 2) Added missing colon after "o" in getopt_long (That means it's expecting an argument) 3) Added more checks for the output directory, if it is writable. Tested those. Change-Id: I4057b81375c4e371390802278afa791e731f012a --- M src/collector.c 1 file changed, 68 insertions(+), 18 deletions(-) Approvals: Diederik: Verified; Looks good to me, approved -- To view, visit https://gerrit.wikimedia.org/r/26554 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4057b81375c4e371390802278afa791e731f012a Gerrit-PatchSet: 2 Gerrit-Project: analytics/webstatscollector Gerrit-Branch: master Gerrit-Owner: Stefan.petrea <[email protected]> Gerrit-Reviewer: Diederik <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
