- remove radio button for validating existing repository but keep performing 
the validation in the back
- make 'clone' button a check button

Signed-off-by: Ioana Grigoropol <ioanax.grigoro...@intel.com>
---
 .../yocto/bc/ui/wizards/install/OptionsPage.java   |   29 ++++----------------
 1 file changed, 5 insertions(+), 24 deletions(-)

diff --git 
a/plugins/org.yocto.bc.ui/src/org/yocto/bc/ui/wizards/install/OptionsPage.java 
b/plugins/org.yocto.bc.ui/src/org/yocto/bc/ui/wizards/install/OptionsPage.java
index 9e94aea..689d2a6 100644
--- 
a/plugins/org.yocto.bc.ui/src/org/yocto/bc/ui/wizards/install/OptionsPage.java
+++ 
b/plugins/org.yocto.bc.ui/src/org/yocto/bc/ui/wizards/install/OptionsPage.java
@@ -54,7 +54,6 @@ public class OptionsPage extends FiniteStateWizardPage {
        private ValidationListener validationListener;
        private Text txtProjectName;
        private Button btnGit;
-       private Button btnValidate;
 
        private RemoteProjectContentsLocationArea locationArea;
 
@@ -67,7 +66,7 @@ public class OptionsPage extends FiniteStateWizardPage {
        public void createControl(Composite parent) {
                top = new Composite(parent, SWT.None);
                top.setLayout(new GridLayout());
-               top.setLayoutData(new GridData(GridData.FILL_BOTH));
+               top.setLayoutData(new GridData(GridData.FILL_HORIZONTAL));
 
                GridData gdFillH = new GridData(GridData.FILL_HORIZONTAL);
 
@@ -97,25 +96,12 @@ public class OptionsPage extends FiniteStateWizardPage {
 
                locationArea = new 
RemoteProjectContentsLocationArea(errorReporter, top, null);
 
-               Group locationValidationGroup = new Group(top, SWT.NONE);
-               locationValidationGroup.setText("Git repository");
-               GridData gd = new GridData(GridData.VERTICAL_ALIGN_END | 
GridData.FILL_HORIZONTAL);
-               locationValidationGroup.setLayoutData(gd);
-               GridLayout gl = new GridLayout(1, false);
-               locationValidationGroup.setLayout(gl);
-
-               btnGit = new Button(locationValidationGroup, SWT.RADIO);
                btnGit.setText("Clone from Yocto Project &Git Repository into 
new location");
                btnGit.setEnabled(true);
                btnGit.setSelection(true);
                btnGit.addSelectionListener(validationListener);
-
-
-               btnValidate = new Button(locationValidationGroup, SWT.RADIO);
-               btnValidate.setText("&Validate existing Git project location");
-               btnValidate.setEnabled(true);
-               btnValidate.setSelection(false);
-               btnValidate.addSelectionListener(validationListener);
+               GridData gd = new GridData(GridData.VERTICAL_ALIGN_END | 
GridData.FILL_HORIZONTAL);
+               btnGit.setLayoutData(gd);
 
                setControl(top);
        }
@@ -165,16 +151,11 @@ public class OptionsPage extends FiniteStateWizardPage {
                String projectPath = projectLoc + separator + getProjectName();
                IHostFile repoDest = RemoteHelper.getRemoteHostFile(connection, 
projectPath, new NullProgressMonitor());
 
-               if(btnValidate.getSelection()) {
+               if(!btnGit.getSelection()) {
                        if (repoDest == null || !repoDest.exists()) {
                                setErrorMessage("Directory " + projectPath + " 
does not exist, please select git clone.");
                                return false;
                        }
-//                     IHostFile gitDescr = 
RemoteHelper.getRemoteHostFile(connection, projectPath + "/.git", new 
NullProgressMonitor());
-//                     if (gitDescr == null || !gitDescr.exists()) {
-//                             setErrorMessage("Directory " + projectPath + " 
does not contain a git repository, please select git clone.");
-//                             return false;
-//                     }
 
                        IHostFile validationFile = 
RemoteHelper.getRemoteHostFile(connection, projectPath + URI_SEPARATOR + 
InstallWizard.VALIDATION_FILE, new NullProgressMonitor());
                        if (validationFile == null || !validationFile.exists()) 
{
@@ -185,7 +166,7 @@ public class OptionsPage extends FiniteStateWizardPage {
                        if (repoDest.exists() && repoDest.isDirectory()) {
                                IHostFile gitDescr = 
RemoteHelper.getRemoteHostFile(connection, projectPath + "/.git", new 
NullProgressMonitor());
                                if (gitDescr != null && gitDescr.exists()) {
-                                       setErrorMessage("Directory " + 
projectPath + " contains a repository, please select validate repository.");
+                                       setErrorMessage("Directory " + 
projectPath + " contains a repository, please choose another location or skip 
cloning the repository.");
                                        return false;
                                }
                        }
-- 
1.7.9.5

_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Reply via email to