lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kinow <>
Subject [GitHub] lucene-solr issue #268: Fix some spell check issues
Date Tue, 31 Oct 2017 08:10:40 GMT
Github user kinow commented on the issue:
    Sure @iorixxx 
    Let's see if I didn't miss anything. So first I found the * files with
`git diff --name-only HEAD~1 | grep TokenizerImpl`
    Then use the file from GitHub master branch.
    git diff --name-only HEAD~1 | grep TokenizerImpl | xargs -I{} git checkout origin/master
    git add .
    git commit -m "Will be rebased..."
    git rebase -i HEAD~2
    # mark with f the "Will be rebased commit"
    And now `git diff --name-only HEAD~1 | grep TokenizerImpl` gives me an empty result.
    Confirm I have the jflex files:
    $git diff --name-only origin/fix-spell-check | xargs -I{} basename {} .java | xargs -I{}
find . -name "{}*"
    Then get the diff (`git diff --patch origin/fix-spell-check `).... and trying to apply
the Java changes to jflex files now... except that... there's no "**occured**" on these jflex
    After a bit of searching here and there, and reading a bit about jflex (had never heard
about it, good to learn something new), looks like the typo is actually in jflex's code.
    Going to push-force the commit that removes the * files.
    Does it make sense? Did I miss something?


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message