yetus-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Smiley (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (YETUS-645) Make leading path component optional in patch files (support IntelliJ patch files)
Date Thu, 26 Jul 2018 01:48:00 GMT

    [ https://issues.apache.org/jira/browse/YETUS-645?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16556768#comment-16556768
] 

David Smiley edited comment on YETUS-645 at 7/26/18 1:47 AM:
-------------------------------------------------------------

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  ;-)   (just having fun with you Steve).
  The actual causes of hiccups can vary.


was (Author: dsmiley):
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)

Mime
View raw message