You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "David Smiley (JIRA)" <ji...@apache.org> on 2018/07/26 01:48:00 UTC
[jira] [Resolved] (YETUS-645) Make leading path component optional
in patch files (support IntelliJ patch files)
[ https://issues.apache.org/jira/browse/YETUS-645?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
David Smiley resolved YETUS-645.
--------------------------------
Resolution: Not A Problem
Thanks so much for you help! So in summary, IntelliJ "create patch" files are fine, and so Steve should stop telling me it's my patch files !/jira/images/icons/emoticons/wink.png! (just having fun with you Steve). The actual causes of hiccups can vary.
> Make leading path component optional in patch files (support IntelliJ patch files)
> ----------------------------------------------------------------------------------
>
> Key: YETUS-645
> URL: https://issues.apache.org/jira/browse/YETUS-645
> Project: Yetus
> Issue Type: Improvement
> Reporter: David Smiley
> Priority: Major
>
> IntelliJ IDEA has a "create patch" feature that generates patch files in a format that isn't supported by some tools like Yetus. In particular, there is no leading "a/" and "b/" in the paths. That appears to be the sole difference requiring these patches to be supported.
>
> See [https://youtrack.jetbrains.com/issue/IDEA-92793]
>
> To work around this, I wrote a one-liner Bash script using SED to insert the "a/" and b/" at the right spots:
> {code:java}
> sed -i '' -e 's/^--- /--- a\//g' -e 's/^+++ /+++ b\//g' "$1"
> {code}
> In this issue I propose that Yetus detect the absence of a/ and b/ and either (a) insert them using a similar script, or (b) toggling the "-p" option when the patch is applied using [git-apply|https://git-scm.com/docs/git-apply] or other tool.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)