Skip to content

Commit

Permalink
Fix a mistake made in #531 (#532)
Browse files Browse the repository at this point in the history
I had initially patterned the new license checks on `OsiOnly` and
`FsfOnly` before realizing that perhaps deny!() didn't make sense here,
and I removed it (or so I thought). It seems I blundered and left one
in--sorry.

I believe this deny!() shouldn't be here.

Apologies for the mistake... I thought I'd double-checked my diff but it
seems I did a poor job of it.

(Also, thanks for catching that other mistake I made in #531 Jake)
  • Loading branch information
zkxs authored Jul 25, 2023
1 parent 41d54d3 commit 2a59de4
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/licenses.rs
Original file line number Diff line number Diff line change
Expand Up @@ -159,8 +159,6 @@ fn evaluate_expression(
BlanketAgreement::Fsf => {
if id.is_fsf_free_libre() {
allow!(IsFsfFree);
} else {
deny!(IsFsfFree);
}
}
BlanketAgreement::OsiOnly => {
Expand Down

0 comments on commit 2a59de4

Please sign in to comment.