Skip to content

Add bee inventory #138

Merged
merged 6 commits into from
Nov 17, 2020
Merged

Add bee inventory #138

merged 6 commits into from
Nov 17, 2020

Conversation

donald
Copy link
Collaborator

@donald donald commented Nov 17, 2020

As discussed today: With this change, make install in mxtools will register as a (fake) bee package mxtools-0.0-0 in the system so that the bee tools like bee query and bee installs collision checks regard files from mxtools, too.

Also some minor fixes for the DESTDIR=... case.

buczek@theinternet:~/git/mxtools (add-bee-inventory)$ sudo make install
buczek@theinternet:~/git/mxtools (add-bee-inventory)$ sudo bee list mxtools
mxtools-0.0-0
buczek@theinternet:~/git/mxtools (add-bee-inventory)$ sudo bee query /usr/sbin/clusterd 
mxtools-0.0-0
  /usr/sbin/clusterd

Register files installed from mxtools with bee so that tools like `bee
query` oder bee's conflict check work. We fake a installed package
mxtools-0.0-0.
@pmenzel
Copy link
Contributor

pmenzel commented Nov 17, 2020

Looks good to me.

Nit: One typo in commit message: apparent.

The explit list of destination directories to create is incomplete which
becomes apparent, if installed into a empty directoy with DESTDIR=...

Create any required directory implicitly.
Don't run update-desktop-database after each installed desktop file but
only once after all files are installed.
After the previous commits, install_desktop_file is now reduced to the
same functionality as install_data, so use this function.
Copy link
Contributor

@wwwutz wwwutz left a comment

Choose a reason for hiding this comment

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

looks good to me, thanks!~

Copy link
Contributor

@david david left a comment

Choose a reason for hiding this comment

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

looks godd to me

@donald donald merged commit 2c0479a into master Nov 17, 2020
@donald donald mentioned this pull request Nov 19, 2020
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants