Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] tests: add tests for boolean dependencies (#164)
Closed #164. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/164#event-1261190197___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] tests: add tests for boolean dependencies (#164)
Please do not test `(X or (Y if Z))`, it may well be that we make this illegal in the future. `(X or (Y if Z))` is the same as `(X or Y or not(Z))`, the `or not(Z)` does not make much sense. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/164#issuecomment-326948738___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] tests: add tests for boolean dependencies (#164)
@ignatenkobrain pushed 2 commits. 45777d2 Add "with" operator to rich dependencies f2d2dd0 tests: add tests for 'with' boolean dependencies -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/rpm-software-management/rpm/pull/164/files/ae56b2ae7b88741eaa36248c992b111460e19c9a..f2d2dd0635ebfc79697d95639c3d878a8cba9fde ___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] tests: add tests for boolean dependencies (#164)
@ffesti, I wrote couple of testcases for "with" op and looks like it doesn't work in some cases ``` 208: unsatisfied WITH requireok 209: satisfied WITH require ok 210: unsatisfied nested WITH-AND require FAILED (rpmdeps.at:851) 211: satisfied nested WITH-AND require FAILED (rpmdeps.at:877) 212: unsatisfied nested WITH-OR require ok 213: satisfied nested WITH-OR requireok 214: unsatisfied nested WITH-IF require ok 215: satisfied nested WITH-IF require - not featured FAILED (rpmdeps.at:983) 216: satisfied nested WITH-IF require - full chain FAILED (rpmdeps.at:1006) 217: unsatisfied nested WITH-IF-ELSE require ok 218: satisfied nested WITH-IF-ELSE require - left clause FAILED (rpmdeps.at:1061) 219: satisfied nested WITH-IF-ELSE require - right clause ok ``` * `WITH-AND` is intentional, but I think we should allow it and internally translate to `WITH-WITH` * `WITH-IF` doesn't understand that right clause is kinda optional * `WITH-IF-ELSE` does something weird wither left clause of `IF` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/164#issuecomment-281974663___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint