[jira] [Created] (NPANDAY-610) ArtifactType as an optional configuration argument

2014-02-28 Thread David Akehurst (JIRA)
David Akehurst created NPANDAY-610:
--

 Summary: ArtifactType as an optional configuration argument
 Key: NPANDAY-610
 URL: https://issues.apache.org/jira/browse/NPANDAY-610
 Project: NPanday
  Issue Type: Improvement
  Components: Maven Plugins
Affects Versions: 1.5.0-incubating
Reporter: David Akehurst


For one of my uses of npanday I need to be able to define the artifactType 
using a configuration argument. (I.e. it is not defined by the package type of 
the pom).

I have implemented this, and attached an svn diff of the trunk to this ticket.
Could we get it into the 1.5.0 release please.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (NPANDAY-610) ArtifactType as an optional configuration argument

2014-02-28 Thread David Akehurst (JIRA)

[ 
https://issues.apache.org/jira/browse/NPANDAY-610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13915578#comment-13915578
 ] 

David Akehurst commented on NPANDAY-610:


===
--- 
plugins/maven-compile-plugin/src/main/java/npanday/plugin/compile/AbstractCompilerMojo.java
 (revision 1572877)
+++ 
plugins/maven-compile-plugin/src/main/java/npanday/plugin/compile/AbstractCompilerMojo.java
 (working copy)
@@ -798,6 +798,14 @@
  */
 
 
+/**
+ * artifactType
+ *
+ * @parameter expression = ${artifactType}
+ */
+protected String artifactType;
+
+
 public void execute() throws MojoExecutionException
 {
 SettingsUtil.applyCustomSettings( getLog(), repositoryRegistry, 
settingsPath );
Index: 
plugins/maven-compile-plugin/src/main/java/npanday/plugin/compile/CompilerMojo.java
===
--- 
plugins/maven-compile-plugin/src/main/java/npanday/plugin/compile/CompilerMojo.java
 (revision 1572877)
+++ 
plugins/maven-compile-plugin/src/main/java/npanday/plugin/compile/CompilerMojo.java
 (working copy)
@@ -104,7 +104,8 @@
 
 compilerConfig.setCommands( getParameters() );
 
-String artifactTypeName = project.getArtifact().getType();
+   
+String artifactTypeName = 
(null!=artifactType)?artifactType:project.getArtifact().getType();
 ArtifactType artifactType = 
ArtifactType.getArtifactTypeForPackagingName( artifactTypeName );
 if ( artifactType.equals( ArtifactType.NULL ) )
 {


 ArtifactType as an optional configuration argument
 --

 Key: NPANDAY-610
 URL: https://issues.apache.org/jira/browse/NPANDAY-610
 Project: NPanday
  Issue Type: Improvement
  Components: Maven Plugins
Affects Versions: 1.5.0-incubating
Reporter: David Akehurst

 For one of my uses of npanday I need to be able to define the artifactType 
 using a configuration argument. (I.e. it is not defined by the package type 
 of the pom).
 I have implemented this, and attached an svn diff of the trunk to this ticket.
 Could we get it into the 1.5.0 release please.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Created] (NPANDAY-611) Linux build of plugins broken by \ in tests

2014-02-28 Thread David Akehurst (JIRA)
David Akehurst created NPANDAY-611:
--

 Summary: Linux build of plugins broken by \ in tests
 Key: NPANDAY-611
 URL: https://issues.apache.org/jira/browse/NPANDAY-611
 Project: NPanday
  Issue Type: Bug
  Components: Maven Plugins
Affects Versions: 1.5.0-incubating
Reporter: David Akehurst


Building the maven plugins on linux breaks when compiling the tests because of 
a few backslashes in the tests.

not been able to test this on windows, but the attached comment of svn diff 
fixes it for linux.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (NPANDAY-611) Linux build of plugins broken by \ in tests

2014-02-28 Thread David Akehurst (JIRA)

[ 
https://issues.apache.org/jira/browse/NPANDAY-611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13915584#comment-13915584
 ] 

David Akehurst commented on NPANDAY-611:


Index: 
components/dotnet-executable/src/test/groovy/npanday/executable/execution/CommandExecutorTest.groovy
===
--- 
components/dotnet-executable/src/test/groovy/npanday/executable/execution/CommandExecutorTest.groovy
(revision 1572887)
+++ 
components/dotnet-executable/src/test/groovy/npanday/executable/execution/CommandExecutorTest.groovy
(working copy)
@@ -89,14 +89,14 @@
 public void testCommandArgWithSpaces()
 throws ExecutionException
 {
-testArgExpansion([a b], 'a b\');
+testArgExpansion([a b], 'a b');
 }
 
 @Test
 public void testCommandArgWithEmbeddedSingleQuotes_middle()
 throws ExecutionException
 {
-testArgExpansion([a ' b], 'a \' b');
+testArgExpansion([a ' b], 'a ' b');
 }
 
 @Test
@@ -417,4 +417,4 @@
 {
return Os.isFamily(Os.FAMILY_WINDOWS);
 }
-}
\ No newline at end of file
+}


 Linux build of plugins broken by \ in tests
 ---

 Key: NPANDAY-611
 URL: https://issues.apache.org/jira/browse/NPANDAY-611
 Project: NPanday
  Issue Type: Bug
  Components: Maven Plugins
Affects Versions: 1.5.0-incubating
Reporter: David Akehurst

 Building the maven plugins on linux breaks when compiling the tests because 
 of a few backslashes in the tests.
 not been able to test this on windows, but the attached comment of svn diff 
 fixes it for linux.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (NPANDAY-611) Linux build of plugins broken by \ in tests

2014-02-28 Thread David Akehurst (JIRA)

[ 
https://issues.apache.org/jira/browse/NPANDAY-611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13915586#comment-13915586
 ] 

David Akehurst commented on NPANDAY-611:


Also, not sure if it is a npanday or a mono bug, but running nunit-console has 
a different command for windows and linux w.r.t v4.

windows nunit-console -v4

linux nunit-console4


 Linux build of plugins broken by \ in tests
 ---

 Key: NPANDAY-611
 URL: https://issues.apache.org/jira/browse/NPANDAY-611
 Project: NPanday
  Issue Type: Bug
  Components: Maven Plugins
Affects Versions: 1.5.0-incubating
Reporter: David Akehurst

 Building the maven plugins on linux breaks when compiling the tests because 
 of a few backslashes in the tests.
 not been able to test this on windows, but the attached comment of svn diff 
 fixes it for linux.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Updated] (NPANDAY-610) ArtifactType as an optional configuration argument

2014-02-28 Thread David Akehurst (JIRA)

 [ 
https://issues.apache.org/jira/browse/NPANDAY-610?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Akehurst updated NPANDAY-610:
---

Attachment: patch.txt

 ArtifactType as an optional configuration argument
 --

 Key: NPANDAY-610
 URL: https://issues.apache.org/jira/browse/NPANDAY-610
 Project: NPanday
  Issue Type: Improvement
  Components: Maven Plugins
Affects Versions: 1.5.0-incubating
Reporter: David Akehurst
 Attachments: patch.txt


 For one of my uses of npanday I need to be able to define the artifactType 
 using a configuration argument. (I.e. it is not defined by the package type 
 of the pom).
 I have implemented this, and attached an svn diff of the trunk to this ticket.
 Could we get it into the 1.5.0 release please.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (NPANDAY-610) ArtifactType as an optional configuration argument

2014-02-28 Thread David Akehurst (JIRA)

[ 
https://issues.apache.org/jira/browse/NPANDAY-610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13915589#comment-13915589
 ] 

David Akehurst commented on NPANDAY-610:


done

 ArtifactType as an optional configuration argument
 --

 Key: NPANDAY-610
 URL: https://issues.apache.org/jira/browse/NPANDAY-610
 Project: NPanday
  Issue Type: Improvement
  Components: Maven Plugins
Affects Versions: 1.5.0-incubating
Reporter: David Akehurst
 Attachments: patch.txt


 For one of my uses of npanday I need to be able to define the artifactType 
 using a configuration argument. (I.e. it is not defined by the package type 
 of the pom).
 I have implemented this, and attached an svn diff of the trunk to this ticket.
 Could we get it into the 1.5.0 release please.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)