gump-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Konstantin Kolinko <knst.koli...@gmail.com>
Subject Re: Gump is building stale Checkstyle sources (and other GitHub projects)
Date Wed, 20 Apr 2016 12:53:59 GMT
2016-04-18 17:37 GMT+03:00 Konstantin Kolinko <knst.kolinko@gmail.com>:
> 2016-04-18 14:15 GMT+03:00 Konstantin Kolinko <knst.kolinko@gmail.com>:
>>
>>
>> Maybe there are some other ideas?
>
>
> I looks that my URL theory is wrong.
>
> git clone  git://github.com/checkstyle/checkstyle
>
> completes successfully
>
>
>> E.g. use "git fetch" instead of "git pull",  as we now follow that
>> command with "git reset --hard".
>> I think original plan was "git reset --hard" followed by "git update".
>>
>> Module-level Work  [5]
>>> update_checkstyle
>>> reset_hard_checkstyle
>>> submodule_update_checkstyle
>>
>> [5] http://vmgump.apache.org/gump/public/checkstyle/
>
>
> Looking closer to the above three update/reset/update commands
>
> $ git pull git://github.com/checkstyle/checkstyle master
> From git://github.com/checkstyle/checkstyle
>  * branch            master     -> FETCH_HEAD
> Already up-to-date.
>
> $ git reset --hard origin/master
> HEAD is now at fe17b53 Issue #3000: removed unused check messages
>
> $ git submodule update --init --recursive
>
>
> I am not sure that the first "git pull" command actually updates origin/master.
> If it does not, then resetting to origin/master actually rolls back to
> old sources.
>
> Usually the pull command is just "git pull" without arguments or "git
> pull origin".
>

<git debug="true" /> is working now, and the results for Checkstyle project are:

http://vmgump.apache.org/gump/public/checkstyle/

1. update_checkstyle

[[[

Command Line

git pull git://github.com/checkstyle/checkstyle master

Output

>From git://github.com/checkstyle/checkstyle
 * branch            master     -> FETCH_HEAD
Updating f9ce38f..d3bafb1
Fast-forward
 src/xdocs/writingjavadocchecks.xml.vm | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

]]]

2. reset_hard_checkstyle

[[[

Command Line

git reset --hard origin/master

Output

HEAD is now at f9ce38f Issue #3105: fixed errors in indentation input files

]]]

3. submodule_update_checkstyle

[[[

Command Line

git submodule update --init --recursive

Output

No output to stdout/stderr from this command.

]]]


Looking at project history on GitHub,
https://github.com/checkstyle/checkstyle/commits/master

f9ce38fba4428b1771dd0e405b450d3b8b46bae3 (f9ce38f)  is a commit from
Apr 18, 2016
d3bafb185b09d28c75ee3d160abe710a43b00d5f (d3bafb1) is a commit from Apr 20, 2016

So "git pull" merged changes between these two commits into current
branch,  then "git reset" resulted in a revert to  f9ce38f, thus
resulting in stale sources.


That "git pull URL branch" command has to be replaced with something else.

Best regards,
Konstantin Kolinko

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@gump.apache.org
For additional commands, e-mail: general-help@gump.apache.org


Mime
View raw message