You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by GitBox <gi...@apache.org> on 2019/02/08 17:53:41 UTC

[GitHub] d3zd3z opened a new pull request #268: Fix case on 'sirupsen' dependency

d3zd3z opened a new pull request #268: Fix case on 'sirupsen' dependency
URL: https://github.com/apache/mynewt-newt/pull/268
 
 
   From the readme for the github.com/sirupsen/logrus dependency:
   
       **Seeing weird case-sensitive problems?** It's in the past been possible to
       import Logrus as both upper- and lower-case. Due to the Go package environment,
       this caused issues in the community and we needed a standard. Some environments
       experienced problems with the upper-case variant, so the lower-case was decided.
       Everything using `logrus` will need to use the lower-case:
       `github.com/sirupsen/logrus`. Any package that isn't, should be changed.
   
   Having this with the wrong case prevents 'newt' from being used as a
   dependency with go modules.
   
   This fix changes the dependency case in all of the source files (as well
   as a few other files), fixes the resulting checksum in the go.sum file,
   and renames the directory in the vendor tree.  It otherwise should have
   no effect.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services