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
databricks-connect: init at 7.1.0 #96638
Conversation
is this meant to be used an application or python package? (do you just do |
according to https://docs.databricks.com/dev-tools/databricks-connect.html it looks like it's meant to be used as a cli tool |
if it's only meant to be used from the cli, please use https://github.com/NixOS/nixpkgs/blob/master/doc/languages-frameworks/python.section.md#buildpythonapplication-function as this allows for separation of python environments EDIT: |
You use it as a python package. But have some applications for setup. Normal user case is:
|
8eb4a51
to
eecfefd
Compare
Please follow CONTRIBUTING.md and manual#submitting-changes-making-patches and squash the fix-up commits. in this case, you should have two commits:
|
44e9626
to
5eedc11
Compare
Have now clean up the commits, so now there are three commits:
|
@jonringer thanks for feedback and help so fare. I have one problem/question left. If I make a nix-shell with the following script:
then using the python package But if I install an environment with
Then I have to manually set the environment variable Do you have any clue what the problem can be? I think it can be something with the wrapping. |
UPDATE
Then everything works fine. But as you can see I have to add |
sorry, I sorted python-packages, can you rebase? :) |
oh, i forgot to respond
For it to appear in a nix-shell, you would need to do:
however, inputsFrom has other implications, it's easier to just add jdk like you did above |
5eedc11
to
2422a8d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
shows usage
https://github.com/NixOS/nixpkgs/pull/96638
3 packages built:
python27Packages.databricks-connect python37Packages.databricks-connect python38Packages.databricks-connect
thanks @kfollesdal for opening your first nixpkgs PR :) |
Thanks @jonringer, I hope I can contribute more. Regarding the
But if I install
Then it do not work as expected. But if I add
Then everything works fine as in the |
I think your issue is |
Ok, thanks. I will look more into it :-) |
Python package databricks-connect is missing in nixpkgs. Used this as an opportunity to try make my first nix expression and nixpkgs.
Very happy for reviews and questions.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)