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

ignore temporary files #282

Closed
wants to merge 1 commit into from
Closed

Conversation

cblp
Copy link

@cblp cblp commented Nov 1, 2016

Temporary files are being regenerated often and polluting git status.

Change-Id: Ib6635964ab235da7e0784d3a9bd7290afa96d38a
@ploxiln
Copy link
Contributor

ploxiln commented Nov 1, 2016

worth noting that these files are not present in the master branch, and may be intentionally kept in the maintenance 0.12 branch - ef22400

@cblp
Copy link
Author

cblp commented Nov 1, 2016

Yes, it may be something wrong with my build environment configuration.

@ploxiln
Copy link
Contributor

ploxiln commented Nov 1, 2016

This is probably autotools "maintainer mode". If you get the source extracted from the tarball, the modification times of the files are such that configure has the same timestamp as configure.ac, or newer, same for Makefile.in and Makefile.am etc. But if you clone/checkout from git, the files are created in an arbitrary order, and on a filesystem with better-than second precision, it's likely that one of these files will be older than the file it is derived from. Then autoconf etc. will try to re-generate it.

One fix for this is to touch all files with a specific timestamp argument so they all have the same modification time. I think the ./configure --disable-dependency-tracking option could also work.

@hawicz
Copy link
Member

hawicz commented Nov 3, 2016

Yes, those files are intentionally added on release branches. Not merging this change.

@hawicz hawicz closed this Nov 3, 2016
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

Successfully merging this pull request may close these issues.

None yet

3 participants