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

hw/nxagent: Drop support for MCOPGLOBALS atom needed by KDE 3's ancie… #408

Open
wants to merge 1 commit into
base: 3.6.x
Choose a base branch
from

Conversation

sunweaver
Copy link
Member

…nt ArtsD sound server.

@sunweaver sunweaver added this to the 3.6.0.0 milestone Mar 22, 2017
@sunweaver sunweaver self-assigned this Mar 22, 2017
@sunweaver sunweaver requested a review from uli42 March 22, 2017 19:16
@sunweaver
Copy link
Member Author

sunweaver commented Mar 22, 2017

@uli42: Can you take a quick look at #408? The removal (reason) should be self-explanatory.

Copy link
Member

@uli42 uli42 left a comment

Choose a reason for hiding this comment

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

Isn't that commit "remove NXAGENT_ARTSD support?

However, I am still using KDE3 (well, Trinity), so I'd rather not drop it. The reason I am using it is that konsole is much faster in KDE3 than in KDE4. On the other hand I am not using sound at all...

@sunweaver
Copy link
Member Author

sunweaver commented Mar 22, 2017 via email

@uli42
Copy link
Member

uli42 commented Mar 23, 2017

As said above I do not use sound for NX. But in the TDE repo there's some reference to artsd: http://mirror.git.trinitydesktop.org/cgit/tdemultimedia/tree/

Also see here: http://www.giustetti.net/wiki/index.php?title=En/TDE_arts

@sunweaver sunweaver modified the milestones: 3.7.0.x, 3.6.0.0 Mar 24, 2017
@sunweaver
Copy link
Member Author

Let's postpone this until a later point in time. MCOPGLOBALS code does not hurt, and we don't want to stress the Trinity users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants