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.

Sign up for GitHub

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

Jump to bottom

arch-defs not downloading the latest version of prjxray-db when building #2351

Closed
MidsummerNight opened this issue Dec 20, 2021 · 6 comments
Closed

arch-defs not downloading the latest version of prjxray-db when building #2351

MidsummerNight opened this issue Dec 20, 2021 · 6 comments

Comments

@MidsummerNight
Copy link

Right now after running make env in symbiflow-arch-defs the prjxray-db downloaded is quite outdated (notably missing the new spartan7 info, which I intend to use for adding Spartan-7 support to arch-defs).

How can I let arch-defs always pull the latest prjxray-db data upon building? I've been trying to track down why make env keeps downloading the old version but so far no luck. @acomodi @mithro @mkurc-ant @kgugala Any ideas? Thanks!
@acomodi
Copy link
Contributor

acomodi commented Dec 20, 2021

Hi @MidsummerNight, in theory the package version specified in the conda-lock is pointing to the version that includes the spartan7 changes. Maybe a clean build might solve the issue?

@MidsummerNight
Copy link
Author

@acomodi Yeah that's what's bothering me. I did clone a clean new copy of arch-defs from Github to a brand new directory, then ran make env in that new directory, but no, the new prjxray-db with spartan7 was not downloaded. Can confirm not even a single spartan7 folder exists:

me@me-machine:~/arch-defs$ find . -type d -name spartan7
me@me-machine:~/arch-defs$ find . -type d -name artix7
./symbiflow-arch-defs/third_party/symbiflow-xc-fasm2bels/third_party/prjxray-db/artix7
./symbiflow-arch-defs/third_party/symbiflow-xc-fasm2bels/third_party/prjxray/database/artix7
./symbiflow-arch-defs/third_party/symbiflow-xc-fasm2bels/third_party/prjxray/settings/artix7
./symbiflow-arch-defs/third_party/symbiflow-xc-fasm2bels/third_party/prjxray/fuzzers/074-dump_all/ignored_wires/artix7
./symbiflow-arch-defs/third_party/symbiflow-xc-fasm2bels/third_party/prjxray/third_party/display_port/src/artix7
./symbiflow-arch-defs/third_party/prjxray/database/artix7
./symbiflow-arch-defs/third_party/prjxray/settings/artix7
./symbiflow-arch-defs/third_party/prjxray/fuzzers/074-dump_all/ignored_wires/artix7
./symbiflow-arch-defs/third_party/prjxray/third_party/display_port/src/artix7
./symbiflow-arch-defs/xc/xc7/archs/artix7
./symbiflow-arch-defs/build/xc/xc7/archs/artix7
./symbiflow-arch-defs/env/downloads/conda-pkgs/prjxray-db-0.0_257_g0a0adde-20211214_154543/share/symbiflow/prjxray-db/artix7
./symbiflow-arch-defs/env/conda/envs/symbiflow_arch_def_base/share/symbiflow/prjxray-db/artix7
me@me-machine:~/arch-defs$ ls -alk
total 12
drwxrwxr-x  3 me me 4096 Dec 20 16:32 .
drwxr-xr-x 51 me me 4096 Dec 20 17:15 ..
drwxrwxr-x 18 me me 4096 Dec 20 16:59 symbiflow-arch-defs
me@me-machine:~/arch-defs$

@acomodi
Copy link
Contributor

acomodi commented Dec 20, 2021

@MidsummerNight Right, discovered the issue and opened a PR in the conda build script: hdl/conda-eda#151

@MidsummerNight
Copy link
Author

@acomodi Okay thanks! Once your PR gets merged into conda-eda I should be able to get the prjxray-db with spartan7 info by doing a clean download and build for arch-defs right?

@acomodi
Copy link
Contributor

acomodi commented Dec 20, 2021 •
edited by unfurl-links bot

Might take a bit more, as the flow is the following:

  • A scheduled GH action builds the conda packages once every day, and uploads them
  • Another scheduled GH action in arch-defs checks for new conda-packages and opens a PR to update the packages to the latest (e.g. [BOT] Conda Lock Update #2345)
  • When the BOT PR gets merged, the package will be available.

You can also check probably tomorrow the latest package from conda https://anaconda.org/LiteX-Hub/prjxray-db/files, if it is there, you can just grab it and update the conda_lock.yml file.

@MidsummerNight MidsummerNight mentioned this issue Dec 20, 2021
Closed
@MidsummerNight
Copy link
Author

Conda now downloads the latest prjxray-db with spartan7. I think this issue can be closed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MidsummerNight @acomodi