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

Files in the git repository #1

Open
ppisar opened this issue Nov 30, 2017 · 3 comments
Open

Files in the git repository #1

ppisar opened this issue Nov 30, 2017 · 3 comments

Comments

@ppisar
Copy link

ppisar commented Nov 30, 2017

I have two suggestions to how the git repository is organized:

(1) There is toplevel rwhoisd directory. It's unnecessary. Could you move all the file one level up and remove the rwhoisd directory so that configure.ac is directly at the top of the git repository?

(2) The repository tracks generated file like "configure" file. This is wrong because once you touch configure.ac, the configure file will become invalid and it changes a lot and that will produce hard to read patches as they will contain generated lines. Please remove the generated files (./configure, ./regexp/Makefile) from the git repository.

@Schievel1
Copy link

It is absolutely normal to have configure scripts and deliver them with the source. Users and maintainers usually use the configure script that comes with the source and only if its necessary run autoreconf etc.

@ppisar
Copy link
Author

ppisar commented Dec 1, 2023

It's normal to have pregenerated configure scripts in release tar archives. But not in git source tree. This is what "make dist" does.

@Schievel1
Copy link

I see your point that configure scripts shouldn’t be in git source trees, since they should be generated and if you change something that requires a change in configure you can easily forget to do that.

Nevertheless it is still common to have configure scripts in git source repositories. I could give you tons of examples, but here’s just a popular one: https://github.com/vim/vim/blob/master/src/auto/configure

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

2 participants