MIPT-ILab / mipt-mips

Cycle-accurate pre-silicon simulator of RISC-V and MIPS CPUs

Home Page:http://mipt-ilab.github.io/mipt-mips/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add Git Submodules manual to Wiki

pavelkryukov opened this issue · comments

We need a very good explanation of Git Submodules.
As a experiment, you may try to update our binutils submodule to version 2.29.1

Add wiki page as Git Submodules

A nice page!

  • I see you used some text from more detailed GitHub manual — can you please add a link to it and to other materials you used?
  • Please, add links to the page in _Sidebar and Home
  • I would like to see a description of the following flows:
  • - «if I want to change some trace and then to fix some code in the main repository, what should I do?»
  • - GNU Binutils/GoogleTest version update

GNU Binutils/GoogleTest version update

Just to clarify, we usually update to the latest tagged version (like binutils_2.29_1).

Corrected your comments, add links to sidebar and page home, near link about using git.
Try to update binutils, but I can't build it in Travic Ci
This is log: https://travis-ci.org/inedostoev/mipt-mips

They've added some code which Clang does not like. We may switch Binutils build back to GCC7 by following steps:

  • Revert my commit 29ca18d
  • Add gcc@7 to OS X requirements:
- if [ "$TRAVIS_OS_NAME" == "osx" ]; then brew install libelf zlib; fi
+ if [ "$TRAVIS_OS_NAME" == "osx" ]; then brew install libelf zlib gcc@7; fi

Actually it is a good opportunity for you to report the bug to GNU Binutils community. Their bugtracker is here: https://sourceware.org/bugzilla/.

Ok, it seems there is an alternative solution to remove error for OS X (and maybe for Clang): add --disable-werror option to Binutils configure script

BTW, GNU folks claim they won't fix the problem with sbrk deprecation on OS X: https://sourceware.org/bugzilla/show_bug.cgi?id=16644

But the problem below should be reported. Undefined behavior is a very dangerous thing. This article demonstrates an interesting case, please check it.

bfd.c:519:21: error: passing an object that undergoes default argument promotion to 'va_start' has undefined behavior [-Werror,-Wvarargs]`
      va_start (ap, error_tag);

Update binutils to realese ver. 2.29_1
GoogleTest was already updated to 1.8, it is last version

Ok, thanks. What about

I would like to see a description of the following flows:
...
GNU Binutils/GoogleTest version update

?

Oh, sorry. I thought, that it was task only to update this submodule.
Now I add description of update Binutuls, GoogleTest and the rest of the submodules.

Great, thank you!