D29787: Fix krununittest

2020-05-17 Thread Ahmad Samir
ahmadsamir added a comment. In D29787#672139 , @dfaure wrote: > Let's hope the CI has xterm installed I doubt it. Maybe we'll need to pick "ls" instead, even if that will look weird. The CI seems to have xterm available as the unit

D29787: Fix krununittest

2020-05-16 Thread Ahmad Samir
This revision was automatically updated to reflect the committed changes. Closed by commit R241:ef7708d1f5ec: Fix krununittest (authored by ahmadsamir). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29787?vs=82975=83011 REVISION DETAIL

D29787: Fix krununittest

2020-05-16 Thread David Faure
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Let's hope the CI has xterm installed I doubt it. Maybe we'll need to pick "ls" instead, even if that will look weird. REPOSITORY R241 KIO BRANCH l-terminal (branched from master)

D29787: Fix krununittest

2020-05-15 Thread Ahmad Samir
ahmadsamir created this revision. ahmadsamir added reviewers: Frameworks, dfaure. Herald added a project: Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY After 6452a34cf01d03d316 ,