Author: khmarbaise Date: Sun May 7 13:24:25 2017 New Revision: 1794196 URL: http://svn.apache.org/viewvc?rev=1794196&view=rev Log: [MSHARED-637] Remove IOException from ProjectDeployer cause it's not thrown by the code o Fixed duplicated license header ProjectDeployer. o Changed the deploy() method and removed IOException and added IllegalArgumentException
Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/ProjectDeployer.java maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/ProjectDeployer.java URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/ProjectDeployer.java?rev=1794196&r1=1794195&r2=1794196&view=diff ============================================================================== --- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/ProjectDeployer.java (original) +++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/ProjectDeployer.java Sun May 7 13:24:25 2017 @@ -19,27 +19,6 @@ package org.apache.maven.shared.project. * under the License. */ -import java.io.IOException; - -/* - * Licensed to the Apache Software Foundation (ASF) under one - * or more contributor license agreements. See the NOTICE file - * distributed with this work for additional information - * regarding copyright ownership. The ASF licenses this file - * to you under the Apache License, Version 2.0 (the - * "License"); you may not use this file except in compliance - * with the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, - * software distributed under the License is distributed on an - * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY - * KIND, either express or implied. See the License for the - * specific language governing permissions and limitations - * under the License. - */ - import org.apache.maven.artifact.repository.ArtifactRepository; import org.apache.maven.project.ProjectBuildingRequest; import org.apache.maven.shared.project.NoFileAssignedException; @@ -75,12 +54,12 @@ public interface ProjectDeployer * @param buildingRequest {@link ProjectBuildingRequest} * @param request {@link ProjectDeployerRequest} * @param artifactRepository {@link ArtifactRepository} - * @throws IOException In case of problems related to checksums. * @throws NoFileAssignedException In case of missing file which has not been assigned to project. + * @throws IllegalArgumentException in case of artifact is not correctly assigned. */ - //FIXME: Need to check to remove IOException cause it looks like it's not being thrown by code. void deploy( ProjectBuildingRequest buildingRequest, ProjectDeployerRequest request, ArtifactRepository artifactRepository ) - throws IOException, NoFileAssignedException; + throws NoFileAssignedException, IllegalArgumentException; + } Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java?rev=1794196&r1=1794195&r2=1794196&view=diff ============================================================================== --- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java (original) +++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java Sun May 7 13:24:25 2017 @@ -20,7 +20,6 @@ package org.apache.maven.shared.project. */ import java.io.File; -import java.io.IOException; import java.util.ArrayList; import java.util.Collection; import java.util.List; @@ -66,7 +65,7 @@ public class DefaultProjectDeployer */ public void deploy( ProjectBuildingRequest buildingRequest, ProjectDeployerRequest request, ArtifactRepository artifactRepository ) - throws IOException, NoFileAssignedException + throws NoFileAssignedException, IllegalArgumentException { Artifact artifact = request.getProject().getArtifact();