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

New Package: Google Cloud SDK #4528

Closed
wants to merge 6 commits into from

Conversation

aaronlevin
Copy link
Contributor

This is my first non-Haskell package. Google manages this package strangely: it's not version'd, uses $HOME during installation, etc. Hopefully I did the right things (I tried to look at other similar packages for inspiration).


meta = {
homepage = https://cloud.google.com/sdk/;
description = "Google Cloud SDK contains tools and libraries that enable you to easily create and manage resources on Google Cloud Platform, including App Engine, Compute Engine, Cloud Storage, BigQuery, Cloud SQL, and Cloud DNS.";
Copy link
Member

Choose a reason for hiding this comment

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

Descriptions should be very concise, e.g. Tools and libraries for managing Google Cloud Platform resources.

@aaronlevin
Copy link
Contributor Author

@edolstra I made the updates. Looking into why Travis CI is reporting a failure (something about python 3.4 which is odd because I specify python 2.7). Currently compiling locally but it's doing the jdk from scratch so it'll take a bit.

@aaronlevin
Copy link
Contributor Author

@edolstra this seems to be building now.

@aristidb
Copy link
Contributor

After fixing the SHA256:

Traceback (most recent call last):
  File "bin/bootstrapping/install.py", line 368, in <module>
    main()
  File "bin/bootstrapping/install.py", line 347, in main
    pargs.additional_components)
  File "bin/bootstrapping/install.py", line 310, in Install
    components = InstallComponents(to_install)
  File "bin/bootstrapping/install.py", line 332, in InstallComponents
    components.update(component_ids=component_ids, allow_no_backup=True)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/calliope/frontend.py", line 235, in __call__
    post_run_hooks=None, kwargs=kwargs)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/calliope/frontend.py", line 274, in _Execute
    pre_run_hooks=pre_run_hooks, post_run_hooks=post_run_hooks)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/calliope/backend.py", line 882, in Run
    result = command_instance.Run(args)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/lib/googlecloudsdk/gcloud/sdktools/components/update.py", line 65, in Run
    args.component_ids, allow_no_backup=args.allow_no_backup)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/update_manager.py", line 407, in Update
    install_state, diff = self._GetStateAndDiff()
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/update_manager.py", line 211, in _GetStateAndDiff
    latest_snapshot = self._GetLatestSnapshot()
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/update_manager.py", line 207, in _GetLatestSnapshot
    return snapshots.ComponentSnapshot.FromURLs(*self.__url.split(','))
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/snapshots.py", line 123, in FromURLs
    for url in urls]
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/snapshots.py", line 153, in _DictFromURL
    raise URLFetchError(extra_repo=extra_repo)
googlecloudsdk.core.updater.snapshots.URLFetchError: Failed to fetch component listing from server. Check your network settings and try again.

@aaronlevin
Copy link
Contributor Author

@aristidb could be a regression on google's part? I will remove the sha check and retry.

@aaronlevin
Copy link
Contributor Author

@aristidb I've updated SHA. How do we manage artifacts whose SHAs might change?

@aaronlevin
Copy link
Contributor Author

@aristidb I just rebased onto HEAD and ran nix-build -A google-cloud-sdk without any issue.

@aristidb
Copy link
Contributor

There's some discussion about how to deal with such artifacts in #4542

I'm afraid we don't really have a good solution.

@aristidb
Copy link
Contributor

No, this is not a regression. I still have the same problem.

You probably don't use chroot builds, so network accessing during build time work. They're highly impure though.

I can't add packages that don't build in chroot.

@aristidb aristidb added the 9.needs: reporter feedback This issue needs the person who filed it to respond label Oct 25, 2014
@aaronlevin
Copy link
Contributor Author

@aristidb I will look into the issues wrt to chroot. Any specific instructions to force it to build with chroot? (I've not looked too deeply into how exactly chroot works).

@7c6f434c
Copy link
Member

Usually pre-downloading the things that the package tries to download works. Julia programming language is an example, I think.

@7c6f434c
Copy link
Member

@aaronlevin Do I understand it right that this currently works only without chroot builds enabled?

@aaronlevin
Copy link
Contributor Author

@7c6f434c It was working for me until I performed an upgrade and then receive a similar error. So, perhaps we should close this PR without merging.

@domenkozar
Copy link
Member

@aaronlevin cool, let us know if you want to pursue the issue or close PR

@aaronlevin
Copy link
Contributor Author

@iElectric I will close this PR and submit a new one if I have time to figure out the nuances with this package. Thanks!

@aaronlevin aaronlevin closed this Jan 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9.needs: reporter feedback This issue needs the person who filed it to respond
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants