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

Update instructions for getting tar-files #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tomtrafford
Copy link
Contributor

No description provided.

@Araneidae
Copy link
Contributor

Please reference issue #62 as I think we need to discuss how to proceed in a bit more detail.

Unfortunately I think there are still several issues that need to be addressed:

  • First of all, what do we (or other users) do when any of the tar file sources is unavailable? This does happen: links change, sites move, or even sometime sites go offline temporarily or even permanently. This is why I have always urged for a curated collection of definitive copies of the tar files (and this is why the MD5 sums are always checked).
  • Secondly, the user needs to be advised that there are now two targets available (zynq and zynqmp, I think). These need to be mentioned in the documentation.
  • Thirdly, the list of MD5 sums in the document is good ... but it's a headache to maintain, and I can see already that it is out of date, it doesn't match the list of files being downloaded by the referenced script.
  • Finally, I see that we do generate a sensible looking collection of assets in the Releases page. How is this generated? This should definitely be referenced.

Copy link
Contributor

@Araneidae Araneidae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As noted this change doesn't really address the underlying problems. See #62 for discussion.

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

Successfully merging this pull request may close these issues.

2 participants