Bug 807507 - yast2-nis-client doesn't handle systemd service files propery
Summary: yast2-nis-client doesn't handle systemd service files propery
Status: RESOLVED FIXED
: 804894 (view as bug list)
Alias: None
Product: openSUSE 12.3
Classification: openSUSE
Component: YaST2 (show other bugs)
Version: RC 2
Hardware: Other Other
: P5 - None : Major (vote)
Target Milestone: ---
Assignee: Lukas Ocilka
QA Contact: Jiri Srain
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-05 14:11 UTC by Ludwig Nussel
Modified: 2013-06-11 07:09 UTC (History)
4 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
patch (2.29 KB, patch)
2013-03-05 14:12 UTC, Ludwig Nussel 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ludwig Nussel 2013-03-05 14:11:31 UTC 
yast nis doesn't handle ypbind and autofs scripts properly as it still reads sysv init script info to determine whether the scripts are enabled.
Comment 1 Ludwig Nussel 2013-03-05 14:12:09 UTC 
Created attachment 528198 [details]
patch
Comment 2 Ludwig Nussel 2013-03-05 14:12:30 UTC 
small correction, the bug is in yast-yast2 Service.ycp
Comment 3 Thomas Göttlicher 2013-03-06 09:10:47 UTC 
Ludwig, thanks for the patch.

Pull request is: https://github.com/yast/yast-yast2/pull/52
Comment 4 Thomas Fehr 2013-03-12 11:16:41 UTC 
Patch seems to be added to repo setting to fixed.
Comment 5 Ludwig Nussel 2013-03-12 12:23:39 UTC 
I don't see an entry from the bug bot. Did anyone submit an actual update for 12.3?
Comment 6 Thomas Fehr 2013-03-12 12:51:17 UTC 
No but the fix will show up in upcoming openSUSE 12.4
Comment 7 Ludwig Nussel 2013-03-12 13:43:13 UTC 
the fix is needed in 12.3
Comment 8 Thomas Fehr 2013-03-12 13:48:00 UTC 
Reassigned to Lukas since there is not maintainer of package yast2.rpm
Comment 9 Ludwig Nussel 2013-03-15 14:02:15 UTC 
The pull request above is incomplete btw. It misses the actual fix.

The Info function needs to be removed in Factory. The warning is just cosmetics.
Comment 10 Ludwig Nussel 2013-03-15 14:46:35 UTC 
forget about the first sentence in comment#9, github web interface fooled me.
Comment 11 Jiří Suchomel 2013-03-15 15:14:40 UTC 
Should we release update of yast2.rpm for 12.3?
Comment 12 Jiří Suchomel 2013-03-15 15:38:57 UTC 
Ludwing, does it actually make sense to use Service::Info or Service::FullInfo now?

They are used at several places from other YaST modules...
Comment 13 Jiří Suchomel 2013-03-15 16:13:06 UTC 
Package for testing:

https://build.opensuse.org/package/show?package=yast2&project=home%3Ajsuchome%3A12.3
Comment 14 Ludwig Nussel 2013-03-18 12:22:37 UTC 
*** Bug 804894 has been marked as a duplicate of this bug. ***
Comment 15 Jiří Suchomel 2013-03-18 12:39:48 UTC 
> osc maintenancerequest home:jsuchome:12.3 yast2 openSUSE:12.3
Using target project 'openSUSE:Maintenance'
159793
Comment 16 Benjamin Brunner 2013-03-18 13:36:04 UTC 
Thanks for your submission. I started an update for openSUSE 12.3. See openSUSE:Maintenance:1461.
Comment 17 Jiří Suchomel 2013-03-18 13:44:55 UTC 
So I think we can close the bug
Comment 18 Swamp Workflow Management 2013-03-25 11:11:56 UTC 
openSUSE-RU-2013:0533-1: An update that has one recommended fix can now be installed.

Category: recommended (moderate)
Bug References: 807507
CVE References: 
Sources used:
openSUSE 12.3 (src):    yast2-2.23.23-1.4.1
Comment 19 Bernhard Wiedemann 2013-05-14 09:00:09 UTC 
This is an autogenerated message for OBS integration:
This bug (807507) was mentioned in
https://build.opensuse.org/request/show/175523 Factory / yast2-tftp-server
Comment 20 Lukas Ocilka 2013-06-11 07:09:07 UTC 
*** Bug 808317 has been marked as a duplicate of this bug. ***