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

Added yast2-sw_single-wrapper to handle rpms in KDE #190

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
5 participants
@ancorgs
Copy link
Contributor

commented Oct 4, 2016

This should fix https://bugzilla.suse.com/show_bug.cgi?id=954143 and https://bugzilla.suse.com/show_bug.cgi?id=1002417 for Leap 42.2 and SLE-12-SP2. Since both are in beta phase, waiting for release managers approval, see comments in first bug report.

@ancorgs ancorgs force-pushed the ancorgs:rpm_kde branch from eda4c6c to 2c10433 Oct 4, 2016

@embar-

This comment has been minimized.

Copy link

commented Oct 4, 2016

We also should add line in .spec file (because yast2-sw_single-wrapper calls xdg-su):

Requires: /usr/bin/xdg-su

@coveralls

This comment has been minimized.

Copy link

commented Oct 4, 2016

Coverage Status

Coverage remained the same at 32.654% when pulling 2c10433 on ancorgs:rpm_kde into 0e938c5 on yast:SLE-12-SP2.

@ancorgs

This comment has been minimized.

Copy link
Contributor Author

commented Oct 4, 2016

Pending of the branching and versioning issues. See thread https://lists.opensuse.org/yast-devel/2016-10/msg00003.html

@@ -101,6 +102,8 @@ This package contains the libraries and modules for software management.

%install
%yast_install
mkdir -p %{buildroot}%{_bindir}

This comment has been minimized.

Copy link
@lslezak

lslezak Apr 4, 2017

Member

Actually why the wrapper script cannot be part of the tarball and installed the usual way? With this approach the wrapper will not be installed when you run rake install from the sources. That will be confusing, if you update the file locally your changes will not take effect...

@lslezak

This comment has been minimized.

Copy link
Member

commented Apr 4, 2017

BTW is SP2 still relevant? Maybe master is enough...

@teclator

This comment has been minimized.

Copy link
Contributor

commented Apr 4, 2017

Read reason to close at #195

@teclator teclator closed this Apr 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.