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

Support for .DELETE_ON_ERROR ? #46

Closed
tseemann opened this Issue Sep 30, 2017 · 5 comments

Comments

Projects
None yet
3 participants
@tseemann

tseemann commented Sep 30, 2017

I noticed that .DELETE_ON_ERROR is not one of the supported make directives.

Any reason?

@sjackman

@sjackman

This comment has been minimized.

Contributor

sjackman commented Sep 30, 2017

I would really like .DELETE_ON_ERROR too. That and .SECONDARY and .PHONY are the directives that I make regular use of.

@sjackman

This comment has been minimized.

Contributor

sjackman commented Jan 16, 2018

@ihh .DELETE_ON_ERROR is super useful. See https://www.gnu.org/software/make/manual/html_node/Special-Targets.html
For example

all: foo

.DELETE_ON_ERROR:

foo:
	false >$@
❯❯❯ make
false >foo
make: *** [foo] Error 1
make: *** Deleting file `foo'
❯❯❯ cat foo
cat: foo: No such file or directory
❯❯❯ biomake
 false >foo
While building foo: Error 1 executing false >foo
❯❯❯ ls foo
foo
@sjackman

This comment has been minimized.

Contributor

sjackman commented Jan 16, 2018

.SECONDARY is the default behaviour with biomake, which I think is great, so no need for that one.

@sjackman

This comment has been minimized.

Contributor

sjackman commented Jan 18, 2018

Woohoo! Thanks, Ian! It's like Christmas again! 🎄

@tseemann

This comment has been minimized.

tseemann commented Apr 7, 2018

Thanks Ian.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment