napcok/calamares
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[partition] Use convenience-method for running operation

Browse Source
This commit is contained in:
Adriaan de Groot 2021-12-11 16:28:13 +01:00
parent dc7a1e43b7
commit 8bb2c5fc6b
2 changed files with 10 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/modules/partition/jobs/CreatePartitionJob.cpp
View File

@ -11,7 +11,9 @@
#include "CreatePartitionJob.h" #include "CreatePartitionJob.h"
#include "core/KPMHelpers.h"
#include "core/PartitionInfo.h" #include "core/PartitionInfo.h"
#include "partition/FileSystem.h" #include "partition/FileSystem.h"
#include "partition/PartitionQuery.h" #include "partition/PartitionQuery.h"
#include "utils/CalamaresUtilsSystem.h" #include "utils/CalamaresUtilsSystem.h"
@ -273,17 +275,9 @@ CreatePartitionJob::exec()
return createZfs( m_partition, m_device ); return createZfs( m_partition, m_device );
} }
Report report( nullptr );
NewOperation op( *m_device, m_partition ); NewOperation op( *m_device, m_partition );
op.setStatus( Operation::StatusRunning ); return KPMHelpers::execute(
op, tr( "The installer failed to create partition on disk '%1'." ).arg( m_device->name() ) );
QString message = tr( "The installer failed to create partition on disk '%1'." ).arg( m_device->name() );
if ( op.execute( report ) )
{
return Calamares::JobResult::ok();
}
return Calamares::JobResult::error( message, report.toText() );
} }
void void

18
src/modules/partition/jobs/DeletePartitionJob.cpp
View File

@ -10,9 +10,11 @@
*/ */
#include "DeletePartitionJob.h" #include "DeletePartitionJob.h"
#include "core/KPMHelpers.h"
#include "utils/CalamaresUtilsSystem.h" #include "utils/CalamaresUtilsSystem.h"
// KPMcore
#include <kpmcore/core/device.h> #include <kpmcore/core/device.h>
#include <kpmcore/core/partition.h> #include <kpmcore/core/partition.h>
#include <kpmcore/core/partitiontable.h> #include <kpmcore/core/partitiontable.h>
@ -45,7 +47,7 @@ removePartition( Partition* partition )
auto r = CalamaresUtils::System::instance()->runCommand( auto r = CalamaresUtils::System::instance()->runCommand(
{ "sfdisk", "--delete", "--force", partition->devicePath(), QString::number( partition->number() ) }, { "sfdisk", "--delete", "--force", partition->devicePath(), QString::number( partition->number() ) },
std::chrono::seconds( 5 ) ); std::chrono::seconds( 5 ) );
if ( r.getExitCode() !=0 || r.getOutput().contains("failed") ) if ( r.getExitCode() != 0 || r.getOutput().contains( "failed" ) )
{ {
return Calamares::JobResult::error( return Calamares::JobResult::error(
QCoreApplication::translate( DeletePartitionJob::staticMetaObject.className(), "Deletion Failed" ), QCoreApplication::translate( DeletePartitionJob::staticMetaObject.className(), "Deletion Failed" ),
@ -96,17 +98,9 @@ DeletePartitionJob::exec()
return removePartition( m_partition ); return removePartition( m_partition );
} }
Report report( nullptr );
DeleteOperation op( *m_device, m_partition ); DeleteOperation op( *m_device, m_partition );
op.setStatus( Operation::StatusRunning ); return KPMHelpers::execute( op,
tr( "The installer failed to delete partition %1." ).arg( m_partition->devicePath() ) );
QString message = tr( "The installer failed to delete partition %1." ).arg( m_partition->devicePath() );
if ( op.execute( report ) )
{
return Calamares::JobResult::ok();
}
return Calamares::JobResult::error( message, report.toText() );
} }
void void