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.

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

Proposal of changes of the build system and sources #1385

Open
mdziczkowski opened this issue Dec 19, 2021 · 0 comments
Open

Proposal of changes of the build system and sources #1385

mdziczkowski opened this issue Dec 19, 2021 · 0 comments

Comments

@mdziczkowski
Copy link
Contributor

mdziczkowski commented Dec 19, 2021

Hello. I would like to propose following changes in the build system of Pennmush:

  1. Important: Ensure that all required header (.h) and source (.c) files are existing inside the repository

  2. Create a file (for example: BUILDING.md) that would contain a list of the libraries and programs that are required to build of Pennmush and offline instructions for the building.

As example, for the Debian (and deveriative) Linux systems, the main required packages are:

  • build-essentials
  • libssl-dev
  • autoconf
  • m4
  • libtool
  1. Remove from the repository the "configure" file, leaving only the "configure.in".

This proposal will prevent the need to do double edition of the both configure files (configure.in and configure) and with same, doing eventually mistakes.

The "configure" file should become generated each time from the "configure.in" file using the autoconf

  1. Create only one include file to with would refer themself all source files.

Adventages:

  • remove retundancy of the include in the sources and in same perventing eventual errors caused thru the "include errors" (as example: in one source file a header file "x.h" becomes readed in, but in another source fiile, the same header wont't become readed and return an error)

  • possibility to load the headers "on demend", depending if the file has been found

  • as above, depending from the target operating system (on other systems the packages containing this files may have ther names)

  • other, not mentioned yet

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

No branches or pull requests

1 participant