Add built versions of aurpublish files to git ignore#31
Add built versions of aurpublish files to git ignore#31stevenbenner wants to merge 1 commit intoeli-schwartz:masterfrom
Conversation
|
The current gitignore is actually kinda weird because its git origins are actually from when aurpublish was distributed inside my pkgbuilds repository, and the gitignore reflects that by containing content relevant to such. Somewhat in parallel, there is #26 which asks for aurpublish to generate a gitignore for you... Probably you should either delete the current contents before inserting your additions, or move them to some form of documentation-like location. |
|
On a completely unrelated note, this is why real build systems are nice, especially out of source build directories: https://mesonbuild.com/FAQ.html#how-do-i-ignore-the-build-directory-in-my-vcs But it could be considered overkill to add such a dependency for a project that doesn't actually have any build time options or platform-specific behavior. |
This patch includes two changes to clean up the .gitignore file: * Remove old items which are no longer relevant * Add built versions of aurpublish files to the ignore list
666e5f7 to
ef43c5a
Compare
I figured it was something like that. The other stuff didn't make sense to me and mostly dates back to the initial commit. But I was hesitant to nuke it because it's generally a bad idea to delete things just because I don't understand them.
Agreed. Patch updated and description reworded. I chose to delete them. I can see an argument to be made for documenting the practice. But I think that information falls under general git knowledge or PKGBUILD/makepkg knowledge more so than aurpublish. Or at least, it is a topic that I would not expect to see documented in an aurpublish man page. If a little hand-holding is desired then I would a agree that #26 is a good idea. If someone is totally new to this then dropping a reasonable default .gitignore file in the directory when they run
Indeed. If only we had access to an expert on build systems. Perhaps some prolific contributor to one of the most popular build systems on the planet... From a project structure standpoint, something like this traditional structure would probably be a lot cleaner: With an ignored top-level Moving the files around and updating the makefile would be a trivial task. However that's a significant restructuring of the project. It would make it painful to land the other two pull requests if you're considering them.
Meson? Yeah, probably. It's a significant makedepends for a build process which is basically Personally I'd rather have a slightly overkill but easy to reason about solution maintained by an expert than a trivial but obtuse solution slapped in without care and forgotten about. |
These files were always showing up as untracked in
git statuswhile working on the aurpublish script. Since they are not meant to be committed, they should probably be included in the .gitignore list.