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

Raise error when using (target) in %!target and %!includeconf #48

Closed
GoogleCodeExporter opened this Issue Apr 18, 2015 · 1 comment

Comments

Projects
None yet
1 participant
@GoogleCodeExporter

GoogleCodeExporter commented Apr 18, 2015

%!target and %!includeconf are the only commands that do not support the 
optional target specification, such as:

%!csv(html): foo.csv

Where foo.csv will only be included when the target is HTML.

OK
   %!target: html
   %!includeconf: foo.conf

NOT OK
   %!target(txt): html
   %!includeconf(html): foo.conf

Today, the last two commands will be executed as if the user has not specified 
the target. So it will always be true.

This can confuse the user and bring unexpected problems.

Txt2tags should raise an error when user tries to specify a target in commands 
that do not support it.

Original issue reported on code.google.com by aureliojargas@gmail.com on 11 Aug 2010 at 1:45

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 18, 2015

This issue was closed by revision r221.

Original comment by aureliojargas@gmail.com on 11 Aug 2010 at 2:09

  • Changed state: Fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment