Skip to content
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

Gnome mousetweaks4 #16255

Open
wants to merge 1 commit into
base: oi/hipster
Choose a base branch
from

Conversation

isoriano1968
Copy link

Hi,

mousetweaks is one of the many packages to bring GNOME 3 to OI.

Cheers

Iggi

@mtelka
Copy link
Contributor

mtelka commented Feb 24, 2024

You accidentally deleted files in tools directory. Please fix.

@Toasterson
Copy link
Contributor

Toasterson commented Feb 24, 2024

@isoriano1968 asked me via chat if I can cleanup the commit I think I finally found out how :)

# This file sets up the standard, default options and base requirements for
# GNOME userland components.
#
COMPONENT_PROJECT_URL ?= https://www.gnome.org/
Copy link
Contributor

Choose a reason for hiding this comment

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

We already have gnome.mk. No need to introduce something similar.

REQUIRED_PACKAGES += x11/library/libxfixes
REQUIRED_PACKAGES += x11/library/libxtst
# Auto-generated dependencies
REQUIRED_PACKAGES += library/desktop/cairo
Copy link
Contributor

Choose a reason for hiding this comment

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

I do not think it is good idea to bring this from solaris-userland. We already diverged too much and many things makes no sense here when copied verbatim or are even counterproductive.

This is fairly simple component so package it from scratch should be straightforward. And you would also get much smaller Makefile. For example here are duplicate dependencies. Above is COMPONENT_BAID, etc.

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.

3 participants