Summary
sigma uses obsolete find-file signature
Metadata
- Id: e79a3dd5b0cf211e3e7e6a479ec6e4e3bed8ea77
- Trac id: 993
- Type: defect
- Reporter: ckeen
- Owner: iraikov
- Cc:
- Status: closed
- Component: extensions
- Estimated difficulty:
- Resolution: fixed
- Priority: major
- Milestone: someday
- Version: 4.8.x
- Changetime: 2013-03-05 18:48:14 UTC
- Created: 2013-03-04 22:37:14 UTC
- Keywords: sigma find-file
Attachments
- e79a3dd5b0cf211e3e7e6a479ec6e4e3bed8ea77/attachments/find-files.diff
Description
...which silently breaks the current directory "." (and probably others) as the path is not stripped.
An attached patch fixes this but does not check for older chickens. Since this has been deprecated since 4.6.x it may be safe to upgrade.
Changes and comments
[2013-03-04 22:37:31 UTC] ckeen attached find-files.diff (description=#f)
[2013-03-05 01:57:07 UTC] iraikov wrote:
Thanks for the bug report, but the patch was fixes only one invocation of find-files and does not supply the seed argument. I have fixed all three invocations of find-files accordingly and have tagged a new release (2.4).
[2013-03-05 18:48:14 UTC] ckeen changed status from new to closed
[2013-03-05 18:48:14 UTC] ckeen set resolution to fixed
[2013-03-05 18:48:14 UTC] ckeen wrote:
Oh, I am sorry for the incomplete patch! <insert lame excuse about it being late here>. I hope it has not been too inconvenient. According to the documentation the seed keyword argument defaults to '(), which is why I have omitted it.
Thanks for the new release!