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

Makefile.am: fix missing files on "clean" target. #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

emollier
Copy link
Contributor

As seen on Debian bug #1049666, running the "clean" target under certain circumstances may cause the rule to fail with:

rm: cannot remove 'test-reads.fq.gz': No such file or directory

This is because the rm commands are not forcefully run, with flag -f. One way to address this is to use the built-in $(RM) from default make variables, which typically will result in running rm -f instead of plain rm, in turn resulting in no failure if the file hasn't been generated in the first place.

If this variable were to not to be trusted for whatever reason, there is also the option to set it explicitly first, e.g. RM = rm -vf.

As seen on [Debian bug #1049666], running the "clean" target under
certain circumstances may cause the rule to fail with:

	rm: cannot remove 'test-reads.fq.gz': No such file or directory

This is because the rm commands are not forcefully run, with flag -f.
One way to address this is to use the built-in $(RM) from default make
variables, which typically will result in running rm -f instead of
plain rm, in turn resulting in no failure if the file hasn't been
generated in the first place.

If this variable were to not to be trusted for whatever reason, there
is also the option to set it explicitly first, e.g. `RM = rm -vf`.

[Debian bug #1049666]: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1049666

Signed-off-by: Étienne Mollier <emollier@debian.org>
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.

1 participant