TracError: No filespec given

[Ryan J Ollos]

We should return a system_message and log the error rather than raising an exception:

[pid 12594 140268838594304] 2016-08-06 23:07:23,402 Trac[formatter] ERROR: Macro Image(attachment:appveyor-kb-logo.png:wiki:TracDev/AutomaticBuilds) failed:
Traceback (most recent call last):
  File "/usr/local/virtualenv/1.1dev/lib/python2.7/site-packages/trac/wiki/formatter.py", line 802, in _macro_formatter
    return macro.ensure_inline(macro.process(args))
  File "/usr/local/virtualenv/1.1dev/lib/python2.7/site-packages/trac/wiki/formatter.py", line 379, in process
    text = self.processor(text)
  File "/usr/local/virtualenv/1.1dev/lib/python2.7/site-packages/trac/wiki/formatter.py", line 351, in _macro_processor
    text)
  File "/usr/local/virtualenv/1.1dev/lib/python2.7/site-packages/trac/wiki/macros.py", line 677, in expand_macro
    raise TracError(_("No filespec given"))
TracError: No filespec given

[Ryan J Ollos]

Proposed changes in log:rjollos.git:t12555.

[Ryan J Ollos]

It looks like I accidentally pushed this branch to the GitHub mirror. Deleted now.

[Jun Omae]

I think setting invalid url to pushurl could prevent accidental pushing like this:

$ git remote set-url --push mirror no_push
$ git remote -v | grep mirror
mirror  http://svn.edgewall.org/git/trac/mirror (fetch)
mirror  no_push (push)
$ git push mirror x
fatal: 'no_push' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
$ git fetch mirror
$ echo $?
0   # git-fetch works

See also ​http://stackoverflow.com/a/10260389/709074

[Ryan J Ollos]

That works, thanks. Documented in TracTeam/Repositories@13.

[Ryan J Ollos]

Committed to 1.0-stable in r15087, merged to 1.2-stable in r15088, merged to trunk in r15089.