deepin(library): init at 20.8 #210477
deepin(library): init at 20.8 #210477
Conversation
github-actions
bot
added
6.topic: nixos
ofborg
bot
added
8.has: package (new)
|
Result of 2 packages blacklisted:
14 packages built:
|
wineee
force-pushed
the
deepin-lib
branch
2 times, most recently
from
7c9fad8 to
fb4fa4e
Compare
wineee
force-pushed
the
deepin-lib
branch
2 times, most recently
from
efd9d17 to
c5a9fd6
Compare
|
@ofborg build deepin |
pkgs/desktops/deepin/library/deepin-wayland-protocols/default.nix
Outdated
Show resolved
Hide resolved
| propagatedBuildInputs = [ | ||
| dtkcore | ||
| librsvg | ||
| qtimageformats | ||
| freeimage | ||
| ]; |
There was a problem hiding this comment.
Do we really need to do this?
There was a problem hiding this comment.
dtkcore is required in dtkgui.pc, not sure about the rest.
There was a problem hiding this comment.
qtimageformats is a runtime dependency，is uesd for parse dci icons (webp format)
librsvg is runtime dependency uesd by DSvgRenderer
freeimage is runtime dependency uesd by DImageHandler
There was a problem hiding this comment.
This cannot be added via the qt wrapper or another variable?
There was a problem hiding this comment.
QT_PLUGIN_PATH will be set if qtimageformats in buildInputs and wrapQtAppsHook in nativeBuildInputs
if dont use propagatedBuildInputs， we have to set qtimageformats wrapper for all dtk application
| , wrapQtAppsHook | ||
| , glibmm | ||
| , doxygen | ||
| , buildDocs ? false |
There was a problem hiding this comment.
Whats the reason why this is off by default?
| , mtdev | ||
| , cairo | ||
| , xorg | ||
| , waylandSupport ? false |
There was a problem hiding this comment.
wayland should always be compiled in when possible with x11 at the same time.
There was a problem hiding this comment.
The adaptation of DDE to wanyland is not complete yet，Arch also disables this option : https://github.com/archlinux/svntogit-community/blob/27fead70b2c5388907ce5ac4eeaf413bfc8f7ffd/trunk/PKGBUILD#L25
There was a problem hiding this comment.
Maybe we should add a comment explaining this
There was a problem hiding this comment.
Should I remove this option and disable wayland compilation directly？
There was a problem hiding this comment.
By the way, wayland support in deepin may be complete by the end of this year
There was a problem hiding this comment.
na, then keep it like it is.
Description of changes
I'm trying to port Deepin Desktop Environment to NixOS. Resume work of @romildo and (at)worldofpeace on #94870
I created a flake repository for this work. Porting work nearing completion，I think it's time to request a review.
It will need 70+ commits to add DDE for NixOS, Here is the first part, add all libraries needed for DDE.
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)nixos/doc/manual/md-to-db.shto update generated release notes