Hmm, but GH doesn't let me merge despite resolving the conflicts. What a mess,
probably my fault.
I'll handle via #844 separately.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Closed #833.
--
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/833#event-2643037718___
Rpm-maint mailing list
pmatilai approved this pull request.
--
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/833#pullrequestreview-289822247___
Merged the conflict manually.
Thanks.
--
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/833#issuecomment-532626179___
Rpm-maint
@pmatilai pushed 1 commit.
8ac1d0b720b57b02904108b17e90e81d0a48afff Merge branch 'master' into divisionnil
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
Meh. Error messages are not about telling people what they may or may not do
when they *just did*. The message should simply be "error: division by zero"
and no further lecturing on the subject.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly
@pavlinamv pushed 1 commit.
8e919739e766840626d17b0891bdc27ac6be776e Prevent dividing by 0 in the
expression evaluation
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
pmatilai requested changes on this pull request.
See above, nice catch but please fix the error and commit messages.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Nice catch!
It's not a question of allowing or not allowing though, it's simply an error
which we now trap instead of crashing. So drop the "not allowed" from the error
message, and adjust the commit message to that tune too.
--
You are receiving this because you are subscribed to this thread.
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/833
-- Commit Summary --
* Do not allow to divide by 0 in the expression evaluation
-- File Changes --
M rpmio/expression.c (4)
M tests/rpmmacro.at (2)
-- Patch
10 matches
Mail list logo
