KineticPreProcessor / KPP

The KPP kinetic preprocessor is a software tool that assists the computer simulation of chemical kinetic systems
GNU General Public License v3.0
21 stars 11 forks source link

[BUG/ISSUE] bin folder for compilation? #18

Closed jimmielin closed 2 years ago

jimmielin commented 2 years ago

Hi all,

Sorry if I am missing something obvious - when building the KPP code based on the instructions on readthedocs (https://kpp.readthedocs.io/en/latest/usage-details/installation.html), the final command in src/Makefile tries to put the kpp executable into ./bin/kpp:

kpp:    $(OBJS)
        @echo "  "$(CC) $(CC_FLAGS) $(CFLAGS) $(OBJS) -L$(FLEX_LIB_DIR) -lfl -o kpp
        @$(CC) $(CC_FLAGS) $(CFLAGS) $(OBJS) -L$(FLEX_LIB_DIR) -lfl -o kpp
        @mv kpp ../bin

but bin folder isn't present in the repository by default. As a result mv kpp ../bin will instead rename the kpp executable to the name "bin" and put it in the root directory.

This is a minor annoyance but could we just add a "bin" folder and .gitignore the contents by default in the root directory to fix this?

Thanks! Haipeng

laestrada commented 2 years ago

Hi @jimmielin,

I believe that this issue has already been addressed in the dev and main branches by adding a mkdir -p command to ensure that the bin folder is present. Does this work for you?

jimmielin commented 2 years ago

Hi Lucas,

Thank you! I was using an old main branch and the autoreduce branch from it which is why I ran into this. Looks like it is already fixed then :)

laestrada commented 2 years ago

Great, Thanks for closing the issue!