resolves T137
Details
Details
- Reviewers
joebonrichie - Group Reviewers
Triage Team - Maniphest Tasks
- T137: Add Sysbench to Repository
sysbench cpu run
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Just two small nits, otherwise LGTM.
| package.yml | ||
|---|---|---|
| 11 | nit: would prefer a slightly more descriptive description, from their github README: | |
| 18 | nit: we have an %autogen macro which you can use instead. With that you can drop the configure macro and pass the configure options to autogen instead. | |
Comment Actions
With these warnings means I typically won't be able to run arc land on this cleanly
Lint was skipped when generating these changes. Unit tests were skipped when generating these changes.
Comment Actions
I lost .git directory and had to upload raw diff (via phab) to fix provided remarks.
Would it work (for you) if I re-create this patch?