Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Switch to MIT license #20

Merged
merged 33 commits into from
Nov 30, 2018
Merged

Switch to MIT license #20

merged 33 commits into from
Nov 30, 2018

Conversation

TheCount
Copy link

@TheCount TheCount commented Nov 10, 2018 •
edited
Loading

This PR depends on PR #19.

It has only one additional commit, switching out the GPL boilerplate with the MIT license boilerplate, adding it where it is missing.

It also changes the date range from 1997-2015 to 1997-2018.

This implements #18 (comment).

TheCount added 30 commits November 7, 2018 22:27
@TheCount
cocom-config.h.in: remove cruft
2c8595a 
Remove everything that is either not used at all or guaranteed by C90
anyway.
@TheCount
Remove unneeded endianness check
cadb6d6
@TheCount
Ignore out-of-tree build directories
62189e4
@TheCount
Start out-of-tree build with CMake
ccae017
@TheCount
Also build C++ library
6c8f0a1
@TheCount
Remove dependency on cocom-config.h
c4d4fd4
@TheCount
Install headers and static libraries
fef66e4
@TheCount
Move documentation into separate subdirectory
82754ba
@TheCount
Install documentation
f282b20
@TheCount
Move ticker files to separate test directory
c723438 
Tickers are used for testing/benchmarking only, so they can go into a
separate directory.
@TheCount
Build static ticker library for testing
10315e9
@TheCount
Break up compare_parsers.tst.in
8ce4404 
Break up compare_parsers.tst.in into separate test directory
@TheCount
Break up yaep.tst.in
db5ec0d 
Break up yaep.tst.in into individual files and create individual tests.
@TheCount
Explicitly mark generated libraries as static
ccdcb83
@TheCount
Add ticker C++ library
019c2c9
@TheCount
Break up yaep++.tst.in
dc187ca 
Break up yaep++.tst.in into individual files and create individual
tests.
@TheCount
Remove remnants of old build system
1cef350
@TheCount
Test new build system on TravisCI
0356a79
@TheCount
Have C and C++ output matching error messages
943522e
@TheCount
Deduplicate test data files
11e08d4
@TheCount
Deduplicate ANSI C tokenizer
410699a
@TheCount
Remove dead code
e001daa
@TheCount
Bugfix: do not undo visit flag on second traversal
22ac876
@TheCount
C tests: deduplicate parse tree memory management
fe8000c
@TheCount
C tests: deduplicate test_syntax_error()
22418e9
@TheCount
C tests: deduplicate test_read_token()
c977d89
@TheCount
C tests: deduplicate some test mains
dd50c88 
Deduplicate standard yaep_parse_grammar() type tests.
@TheCount
C tests: deduplicate some test mains
05a776d 
Deduplicate standard yaep_read_grammar() type tests.
@TheCount
C tests: deduplicate some test mains
a70cbd2 
Deduplicate most more complex mains.
@TheCount
C++ tests: deduplication
243f8e9 
Perform analogous deduplications as in
52a15f0,
b931e20,
8e37c71,
a649673,
99e6bb4, and
37d7b4a.
@TheCount
Bugfix: properly init OS::initial_segment_length
e346117
@TheCount
Bugfix: avoid out-of-bounds access
c362150 
Avoid out-of-bounds access of symb_code_trans_vect when an unknown
terminal appears in the input (see test19).
@TheCount
Switch to MIT license
ed64360 
Also update year range to 1997-2018.

Implements vnmakarov#18.
@vnmakarov
Copy link
Owner

Thank you for trying to improve YAEP and removing the old code (it is dated when people used RCS :).

My biggest concern with your changes is that you are trying to switch YAEP from autoconf to cmake.

They both have advantages and disadvantages. I really would like to save autoconf. First I tried to apply changes without cmake but I found it is too big work. Now I am thinking to save autoconf and cmake so it could be built by any of these tools.

So I am going to work on this. Your pull request is not forgotten but the changes will be applied in a week or two.

@TheCount
Copy link
Author

Yes, I've seen projects using both CMake and autotools (as in configure.ac and Makefile.am), leaving the choice of which one to use to the user.

This works well, but I'd advise to start from scratch with the autotools. Those .tst.in files were a nightmare to maintain (unless they are autogenerated), and the cocom-config.h didn't do anything (it had some useful tests for ANSI-C features, but these days, ANSI-C is a given; no need to test for it).

@TheCount TheCount mentioned this pull request Nov 18, 2018
Open
@vnmakarov vnmakarov merged commit ed64360 into vnmakarov:master Nov 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TheCount @vnmakarov