Ticket #3739 (new defect)
.gitignore : build process should not left any modified or untracked files
| Reported by: | sorin | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | Future Releases |
| Component: | mc-core | Version: | master |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Branch state: | no branch | Votes for changeset: |
Description
The build process should not left any modified or untracked files.
All temporary or output files should be covered by .gitignore file, but currently there are lots of files left after doing configure and build steps.
While this is fixed we can also introduce an additional build step that check if new files are left over and that would fail the build if unknown files are detected.
Easy to check: do a git check
Note: See
TracTickets for help on using
tickets.
Hmmm, I don't see any non-ignored files in my checkout, maybe you are building in-place instead of doing a proper VPATH build, and so some build artifacts are mixing in.
Anyways, what's much worse, is that there is no single clean documented .gitignore file in the root directory of the repository, but many .gitignore files spread all over the place, partly containing wrong / overlapping / too broad / obsolete entries.
Cleaning this up properly is tons of hard work, but I don't think it's actually useful. Much easier solution would be to declare that anyone not building release versions should do VPATH builds, and if build artifacts leak outside build directory, then it's a problem.
This would allow to simplify and shorten .gitignore patterns dramatically...