Skip to content

Use simpler tokenization#30

Closed
pombredanne wants to merge 6 commits intomasterfrom
29-parse-overlapping-known-symbols
Closed

Use simpler tokenization#30
pombredanne wants to merge 6 commits intomasterfrom
29-parse-overlapping-known-symbols

Conversation

@pombredanne
Copy link
Member

Do not use automaton for tokenization.
As a side effect, spaces are no longer authorized in license symbol.
This corrects #29

Signed-off-by: Philippe Ombredanne pombredanne@nexb.com

Do not use automaton for tokenization.
As a side effect, spaces are no longer authorized in license symbol.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne pombredanne requested a review from tdruez August 2, 2018 07:25
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Also set configure bash in debug mode.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne
Copy link
Member Author

Parsing expressions with spaces is still a valuable feature... So I am closing this in favor of #31

@pombredanne pombredanne closed this Aug 6, 2018
@pombredanne pombredanne deleted the 29-parse-overlapping-known-symbols branch August 6, 2018 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant