Differential D15157

[KMountPoint] Remove AIX support
ClosedPublic
Actions

Authored by bruns on Aug 29 2018, 11:55 PM.

Details

Reviewers
dhaumann
Group Reviewers
Frameworks
Commits
R241:2242013d2118: [KMountPoint] Remove AIX support
Summary

AIX has not seen a release for 3 years and can be considered legacy
and completely untested. Remove supporting code.

Diff Detail

Repository
R241 KIO
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bruns created this revision.Aug 29 2018, 11:55 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptAug 29 2018, 11:55 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
bruns requested review of this revision.Aug 29 2018, 11:55 PM
dhaumann accepted this revision.Aug 30 2018, 8:34 PM
dhaumann added a subscriber: dhaumann.

I guess this makes sense. One could add a #error in case of someone trying to build on aix, but it's probably overkill.

This revision is now accepted and ready to land.Aug 30 2018, 8:34 PM
dhaumann added a comment.Aug 30 2018, 8:35 PM

BTW: did you grep for other locations in kio for "_AIX"? Might make sense.

Closed by commit R241:2242013d2118: [KMountPoint] Remove AIX support (authored by bruns). · Explain WhyAug 30 2018, 9:09 PM
This revision was automatically updated to reflect the committed changes.
bruns added a comment.Aug 30 2018, 9:20 PM

kio is AIX clean now.

Windows :
For WCE (see D15158), there are still several occurences.

Revision Contents
Changeset List

PathPackages
Msrc/core/kmountpoint.cpp (74 lines)

Diff 40718

View Options

src/core/kmountpoint.cpp

Loading...