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

Fix y2log path in popup messages #85

Merged
merged 14 commits into from
Oct 22, 2024

Conversation

scabrero
Copy link
Contributor

Problem

The popup messages refer to /var/log/YaST/y2log when it should be /var/log/YaST2/y2log.

lslezak and others added 14 commits March 26, 2019 12:55
* Update ds installer based on latest python tooling for ds packages, as supported by upstream.
20190702 update kdb support for 389

This is a large rework of kdb for 389, fixing a large number of issues
that prevented the kdb setup from operating correctly. It now is able
to correctly configure and setup Kerberos against a 389 Directory
Server instance.
Signed-off-by: Samuel Cabrero <[email protected]>
Signed-off-by: Samuel Cabrero <[email protected]>
@scabrero scabrero requested a review from dgdavid October 21, 2024 09:57
Copy link
Member

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

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

Wow! This was open too long ago, sorry for overlooking it. Moreover, it's quite similar to #76, isn't it? Not sure why these changes were not in the new versions of the code.

LGTM anyway.

@dgdavid
Copy link
Member

dgdavid commented Oct 22, 2024

Not sure why these changes were not in the new versions of the code.

I have said nothing :) Looks like #76 was open against SP1 few months before and then this merge request for SP4 remain stale for quite a long time.

@scabrero scabrero merged commit 6f76725 into yast:SLE-15-SP4 Oct 22, 2024
@yast-bot
Copy link

✔️ Internal Jenkins job #4 successfully finished
✔️ Created IBS submit request #349325

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.

7 participants