andrius
/
kpmcore
1
0
Fork 0
Code Pull Requests Releases Activity

Use long command line options when possible.

This commit is contained in:
Andrius Štikonas 2016-04-20 16:30:23 +01:00
parent e48abb1794
commit 01c9b9f881
Signed by: andrius
GPG Key ID: C86115C8091792EC
8 changed files with 31 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/fs/btrfs.cpp
View File

@ -126,7 +126,7 @@ bool btrfs::check(Report& report, const QString& deviceNode) const
bool btrfs::create(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("mkfs.btrfs"), QStringList() << QStringLiteral("-f") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("mkfs.btrfs"), QStringList() << QStringLiteral("--force") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
@ -140,7 +140,7 @@ bool btrfs::resize(Report& report, const QString& deviceNode, qint64 length) con
bool rval = false;
ExternalCommand mountCmd(report, QStringLiteral("mount"), QStringList() << QStringLiteral("-v") << QStringLiteral("-t") << QStringLiteral("btrfs") << deviceNode << tempDir.path());
ExternalCommand mountCmd(report, QStringLiteral("mount"), QStringList() << QStringLiteral("--verbose") << QStringLiteral("--types") << QStringLiteral("btrfs") << deviceNode << tempDir.path());
if (mountCmd.run(-1) && mountCmd.exitCode() == 0) {
ExternalCommand resizeCmd(report, QStringLiteral("btrfs"), QStringList() << QStringLiteral("filesystem") << QStringLiteral("resize") << QString::number(length) << tempDir.path());

12
src/fs/linuxswap.cpp
View File

@ -88,9 +88,9 @@ bool linuxswap::resize(Report& report, const QString& deviceNode, qint64 length)
QStringList args;
if (!label.isEmpty())
args << QStringLiteral("-L") << label;
args << QStringLiteral("--label") << label;
if (!uuid.isEmpty())
args << QStringLiteral("-U") << uuid;
args << QStringLiteral("--uuid") << uuid;
args << deviceNode << QString::number(length / 1024);
@ -105,9 +105,9 @@ bool linuxswap::copy(Report& report, const QString& targetDeviceNode, const QStr
QStringList args;
if (!label.isEmpty())
args << QStringLiteral("-L") << label;
args << QStringLiteral("--label") << label;
if (!uuid.isEmpty())
args << QStringLiteral("-U") << uuid;
args << QStringLiteral("--uuid") << uuid;
args << targetDeviceNode;
@ -117,7 +117,7 @@ bool linuxswap::copy(Report& report, const QString& targetDeviceNode, const QStr
bool linuxswap::writeLabel(Report& report, const QString& deviceNode, const QString& newLabel)
{
ExternalCommand cmd(report, QStringLiteral("mkswap"), QStringList() << QStringLiteral("-L") << newLabel << deviceNode);
ExternalCommand cmd(report, QStringLiteral("mkswap"), QStringList() << QStringLiteral("--label") << newLabel << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
@ -149,7 +149,7 @@ bool linuxswap::updateUUID(Report& report, const QString& deviceNode) const
QStringList args;
if (!label.isEmpty())
args << QStringLiteral("-L") << label;
args << QStringLiteral("--label") << label;
args << deviceNode;

6
src/fs/lvm2_pv.cpp
View File

@ -88,7 +88,7 @@ qint64 lvm2_pv::maxCapacity() const
bool lvm2_pv::check(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("lvm"), QStringList() << QStringLiteral("pvck") << QStringLiteral("-v") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("lvm"), QStringList() << QStringLiteral("pvck") << QStringLiteral("--verbose") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
@ -101,13 +101,13 @@ bool lvm2_pv::create(Report& report, const QString& deviceNode) const
bool lvm2_pv::remove(Report& report, const QString& deviceNode) const
{
// TODO: check if PV is a member of an exported VG
ExternalCommand cmd(report, QStringLiteral("lvm"), QStringList() << QStringLiteral("pvremove") << QStringLiteral("-ffy") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("lvm"), QStringList() << QStringLiteral("pvremove") << QStringLiteral("--force") << QStringLiteral("--force") << QStringLiteral("--yes") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
bool lvm2_pv::updateUUID(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("lvm"), QStringList() << QStringLiteral("pvchange") << QStringLiteral("-u") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("lvm"), QStringList() << QStringLiteral("pvchange") << QStringLiteral("--uuid") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
}

4
src/fs/nilfs2.cpp
View File

@ -145,10 +145,10 @@ bool nilfs2::resize(Report& report, const QString& deviceNode, qint64 length) co
bool rval = false;
ExternalCommand mountCmd(report, QStringLiteral("mount"), QStringList() << QStringLiteral("-v") << QStringLiteral("-t") << QStringLiteral("nilfs2") << deviceNode << tempDir.path());
ExternalCommand mountCmd(report, QStringLiteral("mount"), QStringList() << QStringLiteral("--verbose") << QStringLiteral("--types") << QStringLiteral("nilfs2") << deviceNode << tempDir.path());
if (mountCmd.run(-1) && mountCmd.exitCode() == 0) {
ExternalCommand resizeCmd(report, QStringLiteral("nilfs-resize"), QStringList() << QStringLiteral("-v") << QStringLiteral("-y") << deviceNode << QString::number(length));
ExternalCommand resizeCmd(report, QStringLiteral("nilfs-resize"), QStringList() << QStringLiteral("--verbose") << QStringLiteral("--assume-yes") << deviceNode << QString::number(length));
if (resizeCmd.run(-1) && resizeCmd.exitCode() == 0)
rval = true;

10
src/fs/ntfs.cpp
View File

@ -140,19 +140,19 @@ bool ntfs::writeLabel(Report& report, const QString& deviceNode, const QString&
bool ntfs::check(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("ntfsresize"), QStringList() << QStringLiteral("-P") << QStringLiteral("-i") << QStringLiteral("-f") << QStringLiteral("-v") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("ntfsresize"), QStringList() << QStringLiteral("--no-progress-bar") << QStringLiteral("--info") << QStringLiteral("--force") << QStringLiteral("--verbose") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
bool ntfs::create(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("mkfs.ntfs"), QStringList() << QStringLiteral("-f") << QStringLiteral("-vv") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("mkfs.ntfs"), QStringList() << QStringLiteral("--quick") << QStringLiteral("--verbose") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
bool ntfs::copy(Report& report, const QString& targetDeviceNode, const QString& sourceDeviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("ntfsclone"), QStringList() << QStringLiteral("-f") << QStringLiteral("--overwrite") << targetDeviceNode << sourceDeviceNode);
ExternalCommand cmd(report, QStringLiteral("ntfsclone"), QStringList() << QStringLiteral("--force") << QStringLiteral("--overwrite") << targetDeviceNode << sourceDeviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
@ -160,10 +160,10 @@ bool ntfs::copy(Report& report, const QString& targetDeviceNode, const QString&
bool ntfs::resize(Report& report, const QString& deviceNode, qint64 length) const
{
QStringList args;
args << QStringLiteral("-P") << QStringLiteral("-f") << deviceNode << QStringLiteral("-s") << QString::number(length);
args << QStringLiteral("--no-progress-bar") << QStringLiteral("--force") << deviceNode << QStringLiteral("--size") << QString::number(length);
QStringList dryRunArgs = args;
dryRunArgs << QStringLiteral("-n");
dryRunArgs << QStringLiteral("--no-action");
ExternalCommand cmdDryRun(QStringLiteral("ntfsresize"), dryRunArgs);
if (cmdDryRun.run(-1) && cmdDryRun.exitCode() == 0) {

16
src/fs/ocfs2.cpp
View File

@ -46,16 +46,16 @@ ocfs2::ocfs2(qint64 firstsector, qint64 lastsector, qint64 sectorsused, const QS
void ocfs2::init()
{
m_Create = findExternal(QStringLiteral("mkfs.ocfs2"), QStringList() << QStringLiteral("-V")) ? cmdSupportFileSystem : cmdSupportNone;
m_Create = findExternal(QStringLiteral("mkfs.ocfs2"), QStringList() << QStringLiteral("--version")) ? cmdSupportFileSystem : cmdSupportNone;
m_Check = findExternal(QStringLiteral("fsck.ocfs2"), QStringList(), 16) ? cmdSupportFileSystem : cmdSupportNone;
m_Grow = (m_Check != cmdSupportNone && findExternal(QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("-V")) && findExternal(QStringLiteral("debugfs.ocfs2"), QStringList() << QStringLiteral("-V"))) ? cmdSupportFileSystem : cmdSupportNone;
m_Grow = (m_Check != cmdSupportNone && findExternal(QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("--version")) && findExternal(QStringLiteral("debugfs.ocfs2"), QStringList() << QStringLiteral("--version"))) ? cmdSupportFileSystem : cmdSupportNone;
m_Shrink = cmdSupportNone;
// TODO: it seems there's no way to get the FS usage with ocfs2
m_GetUsed = cmdSupportNone;
m_SetLabel = findExternal(QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("-V")) ? cmdSupportFileSystem : cmdSupportNone;
m_UpdateUUID = findExternal(QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("-V")) ? cmdSupportFileSystem : cmdSupportNone;
m_SetLabel = findExternal(QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("--version")) ? cmdSupportFileSystem : cmdSupportNone;
m_UpdateUUID = findExternal(QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("--version")) ? cmdSupportFileSystem : cmdSupportNone;
m_Copy = (m_Check != cmdSupportNone) ? cmdSupportCore : cmdSupportNone;
m_Move = (m_Check != cmdSupportNone) ? cmdSupportCore : cmdSupportNone;
@ -126,7 +126,7 @@ bool ocfs2::create(Report& report, const QString& deviceNode) const
bool ocfs2::resize(Report& report, const QString& deviceNode, qint64 length) const
{
ExternalCommand cmdBlockSize(QStringLiteral("debugfs.ocfs2"), QStringList() << QStringLiteral("-R") << QStringLiteral("stats") << deviceNode);
ExternalCommand cmdBlockSize(QStringLiteral("debugfs.ocfs2"), QStringList() << QStringLiteral("--request") << QStringLiteral("stats") << deviceNode);
qint32 blockSize = -1;
if (cmdBlockSize.run()) {
@ -139,19 +139,19 @@ bool ocfs2::resize(Report& report, const QString& deviceNode, qint64 length) con
if (blockSize == -1)
return false;
ExternalCommand cmd(report, QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("-y") << QStringLiteral("-S") << deviceNode << QString::number(length / blockSize));
ExternalCommand cmd(report, QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("--yes") << QStringLiteral("--volume-size") << deviceNode << QString::number(length / blockSize));
return cmd.run(-1) && cmd.exitCode() == 0;
}
bool ocfs2::writeLabel(Report& report, const QString& deviceNode, const QString& newLabel)
{
ExternalCommand cmd(report, QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("-L") << newLabel << deviceNode);
ExternalCommand cmd(report, QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("--label") << newLabel << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
bool ocfs2::updateUUID(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("-U") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("tunefs.ocfs2"), QStringList() << QStringLiteral("--uuid-reset") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
}

4
src/fs/reiser4.cpp
View File

@ -115,13 +115,13 @@ qint64 reiser4::readUsedCapacity(const QString& deviceNode) const
bool reiser4::check(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("fsck.reiser4"), QStringList() << QStringLiteral("--fix") << QStringLiteral("-y") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("fsck.reiser4"), QStringList() << QStringLiteral("--fix") << QStringLiteral("--yes") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
bool reiser4::create(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("mkfs.reiser4"), QStringList() << QStringLiteral("--yes") << QStringLiteral("--force") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("mkfs.reiser4"), QStringList() << QStringLiteral("--force") << QStringLiteral("--yes") << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
}

6
src/fs/reiserfs.cpp
View File

@ -130,13 +130,13 @@ qint64 reiserfs::readUsedCapacity(const QString& deviceNode) const
bool reiserfs::writeLabel(Report& report, const QString& deviceNode, const QString& newLabel)
{
ExternalCommand cmd(report, QStringLiteral("reiserfstune"), QStringList() << QStringLiteral("-l") << newLabel << deviceNode);
ExternalCommand cmd(report, QStringLiteral("reiserfstune"), QStringList() << QStringLiteral("--label") << newLabel << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
bool reiserfs::check(Report& report, const QString& deviceNode) const
{
ExternalCommand cmd(report, QStringLiteral("fsck.reiserfs"), QStringList() << QStringLiteral("--fix-fixable") << QStringLiteral("-q") << QStringLiteral("-y") << deviceNode);
ExternalCommand cmd(report, QStringLiteral("fsck.reiserfs"), QStringList() << QStringLiteral("--fix-fixable") << QStringLiteral("--quiet") << QStringLiteral("--yes") << deviceNode);
return cmd.run(-1) && (cmd.exitCode() == 0 || cmd.exitCode() == 1 || cmd.exitCode() == 256);
}
@ -164,7 +164,7 @@ bool reiserfs::resize(Report& report, const QString& deviceNode, qint64 length)
bool reiserfs::updateUUID(Report& report, const QString& deviceNode) const
{
const QString uuid = QUuid::createUuid().toString().remove(QRegExp(QStringLiteral("\\{|\\}")));
ExternalCommand cmd(report, QStringLiteral("reiserfstune"), QStringList() << QStringLiteral("-u") << uuid << deviceNode);
ExternalCommand cmd(report, QStringLiteral("reiserfstune"), QStringList() << QStringLiteral("--uuid") << uuid << deviceNode);
return cmd.run(-1) && cmd.exitCode() == 0;
}
}