issues on handling multiple matches delayed target regex

[schettino72]

Refers to tasks missing from PR #58

• need docs
• when --auto-delayed-regex is used, if a target that doesnt exist (or a typo) is passed in the command, it will be silently ignored. It should raise an error saying the target/task/command doesnt exist

[felixfontein]

Isn't the second also true when --auto-delayed-regex is not used?

[schettino72]

Isn't the second also true when --auto-delayed-regex is not used?

Right, it happens as long as the typo matches a regex. With --auto-delayed-regex it always match a regex...

[felixfontein]

I'm not sure if you are aware of this, but the current version still builds too much in some situations. Consider the dodo file (slight modification of doc/tutorial/delayed.py):

import glob

from doit import create_after

@create_after(executed='early', target_regex='.*\.out')
def task_build():
    for inf in glob.glob('*.in'):
        yield {
            'name': inf,
            'actions': ['cp %(dependencies)s %(targets)s'],
            'file_dep': [inf],
            'targets': [inf[:-3] + '.out'],
            'clean': True,
        }

@create_after(executed='build', target_regex='.*')
def task_final():
    return {
        'actions': ['cp a.out a.out.2'],
        'file_dep': ['a.out'],
        'targets': ['a.out.2'],
        'clean': True,
    }

def task_early():
    """a task that create some files..."""
    inter_files = ('a.in', 'b.in', 'c.in')
    return {
        'actions': ['touch %(targets)s'],
        'targets': inter_files,
        'clean': True,
    }

will build a.in, a.out, b.in, b.out, c.in and c.out when running doit run c.out.

[schettino72]

@felixfontein I guess this finally completed. Please check again everything is working as expected.
Thanks for your patience and work on pushing for this feature.

[felixfontein]

Cool! Thanks a lot for implementing most parts of this feature! :)
A quick test also showed that it seems to work as expected. Since there weren't big changes since the last branch I've been using extensively, I guess it should be good by now.

[felixfontein]

Oh, one thing (completely unrelated!) which I just noticed: if I run nikola build bla and it cannot build bla, it complains with

ERROR: cmd `run` invalid parameter: "bla". Must be a task, or a target.
Type "doit list" to see available tasks

Obviously, running doit list won't help, and the command I used wasn't run but build. I know that why this is and how to correctly interpret this because I know how doit and Nikola interact, but I can guess that some Nikola users will be completely confused by this error message. What do you think about a possibility to "adjust" these messages in some way?

[schettino72]

What do you think about a possibility to "adjust" these messages in some way?

Sure. go ahead :D

[schettino72]

I created a new issue to track this #71