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

nxos9kv fails to make with nexus9300v64.10.2.3.F.qcow2 #297

Open
adudek16 opened this issue Sep 7, 2022 · 5 comments
Open

nxos9kv fails to make with nexus9300v64.10.2.3.F.qcow2 #297

adudek16 opened this issue Sep 7, 2022 · 5 comments

Comments

@adudek16
Copy link

adudek16 commented Sep 7, 2022

Makefile:14: warning: overriding recipe for target 'docker-pre-build'
../makefile.include:18: warning: ignoring old recipe for target 'docker-pre-build'
rm -f docker/.qcow docker/.tgz docker/.vmdk docker/*.iso
cp OVMF-pure-efi.fd nxos_config.txt docker
ERROR: Incorrect version string (nexus9300v64.10.2.3.F.qcow2). The regexp for extracting version information is likely incorrect, check the regexp in the Makefile ...

@adudek16
Copy link
Author

adudek16 commented Sep 7, 2022

changing Makefile version line to
VERSION=$(shell echo $(IMAGE) | sed -e 's/.+[^0-9]([0-9].[0-9].[0-9].).qcow2$$/\1/')

works. Added "*"...

@adudek16
Copy link
Author

adudek16 commented Sep 7, 2022

actually, it works somewhat.. It builds but the tag is incorrect.
vrnetlab/vr-nxos9kv 0.2.3.F 572ad0f637f8 6 minutes ago 2.26GB
instead of 10.0.2.3.F

@adudek16
Copy link
Author

So, part of my issue was that sed for osx 11.7 does not like this. However this is still not working..
Maybe use the change suggested by hellt#77 ?

@lemontree61089
Copy link

same issue here, any updates?

@joefodor
Copy link

@adudek16 @lemontree61089
I got around this by just putting in the version for the tag I want and commenting out the original regex
VERSION=16.04.01

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