Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Port to meson #58

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Port to meson #58

wants to merge 13 commits into from

Conversation

cjfloss
Copy link

@cjfloss cjfloss commented Nov 24, 2018

Solves #52

megatux
megatux reviewed Dec 10, 2018
View reviewed changes
Copy link

@megatux megatux left a comment

Choose a reason for hiding this comment

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

Looks ok to me. I uninstalled my copy, rebuild it from source and seems to work fine. The only think I found is the app icon, in dock and app-menu looks like it is not set, maybe it's my fault, I restarted wingpanel after install, just in case.
tomato in menu

data/com.github.tomatoers.tomato.appdata.xml.in Outdated Show resolved Hide resolved
@cjfloss
Copy link
Author

cjfloss commented Dec 11, 2018

Meson was installing the icons to icons/hicolor/icons/* instead of just icons/hicolor/*

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@megatux megatux megatux left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cjfloss @megatux