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 config format to a custom format #2

Open
5 of 8 tasks
TheTechmage opened this issue Oct 15, 2015 · 2 comments
Open
5 of 8 tasks

Switch config format to a custom format #2

TheTechmage opened this issue Oct 15, 2015 · 2 comments
Assignees
@TheTechmage
Labels
cross platform enhancement
Milestone
0.1.2

Comments

@TheTechmage
Copy link
Owner

Renzoku needs to use a custom config file format due to the fact that CentOS 6
cannot compile due to an API change that has occurred within libyaml-cpp.
The same API change makes it so that renzoku only compiles on Debian Jesse and
later versions only. Which basically means that you can only run it if you've
got the latest and greatest operating systems. This is a big no-no in my
opinion.

Rough Outline of Tasks:

  • Build a custom lexer
    • Is it modular (sub-classable, etc)
  • Build a tokenizer
    • get_token returns the lexicon (or token?) type
    • get_value returns the value stored within the CURRENT token
  • Build a parser, using the tokenizer
  • Restructure src/config.cpp to use the custom parser
  • PUSH! =D
@TheTechmage TheTechmage added this to the 0.1.2 milestone Oct 15, 2015
TheTechmage added a commit that referenced this issue Nov 1, 2015
@TheTechmage
Switched from yaml config to libconfig
255ec90 
Switched renzoku from using yaml based configs to libconfig instead. libconfig
is much more in line with the type of config that I was hoping to have renzoku
use in the first place. Now that I have finally decided on an official config
format, I can finally start actually recommending it's use to others since
renzoku should be able to build on other OSs. It used to be that only Linux
Distros with the latest and greatest packages could do so.

This fixes #2 and gets a closer to completing the tasks needed for #1.

Signed-off-by: Colton Wolkins (Ogre) <[email protected]>
@TheTechmage TheTechmage mentioned this issue Nov 1, 2015
Open
@TheTechmage
Copy link
Owner Author

libconfig handles this now instead of libyaml.

@TheTechmage
Copy link
Owner Author

libconfig worked, until I tried compiling it on CentOS 7... Now I'm just going to switch to a whole, home-grown config format. >_<

@TheTechmage TheTechmage reopened this Nov 21, 2015
@TheTechmage TheTechmage self-assigned this Nov 21, 2015
TheTechmage added a commit that referenced this issue Dec 7, 2017
@TheTechmage
Fix parser code so that it works
2939655 
I fixed the parser code so that:
A) It works (for the most part) and,
B) It's easy to use and attach to the main renzoku application.

This is me working towards #2
TheTechmage added a commit that referenced this issue Jan 9, 2018
@TheTechmage
Misc fixes for CentOS 7
74a2c41 
This should fix #2 as far as CentOS 7 goes. Have not tested on CentOS 6 or on
Debian. Those are the next goals.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheTechmage TheTechmage

Labels
cross platform enhancement
Projects
None yet
Milestone
0.1.2
Development

No branches or pull requests
1 participant
@TheTechmage