Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compiling SDK 1.12.3 outputs 1.10 version information (Makefile) #33

Closed
jpsenior opened this issue Aug 29, 2017 · 3 comments
Closed

Compiling SDK 1.12.3 outputs 1.10 version information (Makefile) #33

jpsenior opened this issue Aug 29, 2017 · 3 comments

Comments

@jpsenior
Copy link
Contributor

Steps to reproduce:

  1. wget https://github.com/aristanetworks/EosSdk/archive/v1.12.3.tar.gz
  2. Extract
  3. cd EosSdk-1.12.3
  4. ./build.sh
  5. ..compile..
  6. result binaries, configuration, etc point to version v1.10 -- not v1.12 and are not actually compatible with v1.12 RPMs from arista.com website.

Example:

[jp@centos eos-sdk]$ cd EosSdk-1.12.3/
[jp@centos EosSdk-1.12.3]$ grep VERSION *
<snip>
version.m4:m4_define([_VERSION_],[1.10.0])

Presuming version.m4 should specify 1.12.3 -- not 1.10.0.

@ghost
Copy link

ghost commented Aug 31, 2017

They just replaced the file. They fixed this issue while keeping the same file name for the archive tarball. That's quite unusual but at least it has been fixed.

@avi-kumar
Copy link
Contributor

This file is auto-generated in our build, and wasn't done so properly prior to uploading this release to GitHub. We elected to just modify the file in a new commit, and retag this latest commit as release v1.12.3.
Sorry for any confusion, but as this file should match the release version, we felt that simply modifying the file would be sufficient.

@ruferp
Copy link
Contributor

ruferp commented Aug 31, 2017

We had to replace the tarball and keep the version number, because EOS 4.18.4 had already shipped with that version (in a way, github and EosSdk is less important than arista.com and EOS).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants