Differential D18780

Don't include <fstab.h> anymore
ClosedPublic
Actions

Authored by heikobecker on Feb 6 2019, 8:53 AM.

Details

Reviewers
sandsmark
Commits
R352:70fa1a80ce05: Don't include <fstab.h> anymore
Summary

It was replaced with Solid in
f835801ab377132920c258a10c43874484867dae.
Furthermore, musl currently doesn't provide it (might get implemented
in the future [1]) and filelight thus fails to build against musl.

https://www.openwall.com/lists/musl/2018/03/25/1

Test Plan

Builds fine against musl and glibc

Diff Detail

Repository
R352 Filelight
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
heikobecker created this revision.Feb 6 2019, 8:53 AM
Restricted Application added a subscriber: kde-utils-devel. · View Herald TranscriptFeb 6 2019, 8:53 AM
heikobecker requested review of this revision.Feb 6 2019, 8:53 AM
sandsmark accepted this revision.Feb 9 2019, 12:26 PM
This revision is now accepted and ready to land.Feb 9 2019, 12:26 PM
Closed by commit R352:70fa1a80ce05: Don't include <fstab.h> anymore (authored by heikobecker). · Explain WhyFeb 10 2019, 10:21 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/localLister.cpp (5 lines)

Diff 51305

View Options

src/localLister.cpp

Loading...