Copied over from the old depcheck.
Details
Diff Detail
- Lint
Lint Skipped - Unit
Unit Tests Skipped
| libtaskotron/directives/mash_directive.py | ||
|---|---|---|
| 67 | Are you sure dodelta should be a path to directory? /me does not understand MASH at all, but it is used in True/False fashion in the mash config files. Also, if it should be a directory, than it IMHO does not really make sense to initialize it to False on line #76. | |
mash and createrepo should be added to the list of rpm deps in the readme.
This will need to have some ability to have a bit more control over where the repodata is output - right now it can only put the repodata into the rpmdir.
Overall, it looks pretty good, though
| libtaskotron/directives/mash_directive.py | ||
|---|---|---|
| 65 | arch is also a required input | |
| 67 | Shouldn't dodelta be a boolean? | |
| libtaskotron/directives/mash_directive.py | ||
|---|---|---|
| 67 | /me does not understand mash at all either but it should be True/False. Copy&paste typo... Good catch. | |
For initial code, it looks good to me. Can you file a ticket for controlling the mash output, though? I want to make sure that this can work the way we need it to for depcheck before spending too much more time on it but I also want to be sure that task is recorded
arch is also a required input