Skip to content

KDevelop: Add some propagatedUserEnvPkgs dependencies.#4737

Merged
7c6f434c merged 1 commit intoNixOS:masterfrom
ambrop72:kdevelop-dependencies
Nov 4, 2014
Merged

KDevelop: Add some propagatedUserEnvPkgs dependencies.#4737
7c6f434c merged 1 commit intoNixOS:masterfrom
ambrop72:kdevelop-dependencies

Conversation

@ambrop72
Copy link
Contributor

Now KDevelop should start in a functional form without having to
manually install those.

Now KDevelop should start in a functional form without having to
manually install those.
7c6f434c added a commit that referenced this pull request Nov 4, 2014
KDevelop: Add some propagatedUserEnvPkgs dependencies.
@7c6f434c 7c6f434c merged commit fe62405 into NixOS:master Nov 4, 2014
@edolstra
Copy link
Member

edolstra commented Nov 4, 2014

Is this necessary? All these packages are already added to the system path in modules/services/x11/desktop-managers/kde4.nix.

@lucabrunox
Copy link
Contributor

I don't see kdevelop on other distros depending on those packages. I wouldn't make things that are so easily pluggable to be bloated this way by default.

@7c6f434c
Copy link
Member

7c6f434c commented Nov 4, 2014

Is this necessary? All these packages are already added to the system path in modules/services/x11/desktop-managers/kde4.nix.

Are you implying that only KDE users could possibly use KDevelop for
writing a Qt app?

@ambrop72
Copy link
Contributor Author

ambrop72 commented Nov 4, 2014

Some clarification, these are for:

  • kate: The text editor component, KDevelop doesn't start without it.
  • konsole: The embedded terminal.
  • kde_runtime: The welcome screen, uses Plasma libs.
  • oxygen_icons: Lots of icons not found elsewhere.

It's a shame we have to add these to the profile for KDevelop to pick them up though.

@jgeerds
Copy link
Member

jgeerds commented Nov 5, 2014

Can someone else confirm this? If KDevelop doesn't start without kate installed, we should fix this.

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.

5 participants