only-script for Linux clobbers MVNW_REPOURL
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Affected version
3.3.3
Bug description
Running the Maven Wrapper plugin through the Maven Wrapper's only-script distribution type causes MVNW_REPOURL to be discarded, leaving the distributionUrl written to maven-wrapper.properties at its default value. Consequently, only-script for Linux is unable to update itself in any environment that needs MVNW_REPOURL.
That is, the hypothetical execution
$ MVNW_REPOURL=asdfasdf ./mvnw org.apache.maven.plugins:maven-wrapper-plugin:3.3.3:wrapper -Dtype=only-script -Dmaven=3.9.11 -X
where asdfasdf is a functional mirror should have produced an output similar to
...
[DEBUG] Using repo URL from MVNW_REPOURL environment variable.
[DEBUG] Determined repo URL to use as asdfasdf
[INFO] Configuring .mvn/wrapper/maven-wrapper.properties to use Maven 3.9.11 and download from asdfasdf
but will end up producing
...
[DEBUG] Determined repo URL to use as https://repo.maven.apache.org/maven2
[INFO] Configuring .mvn/wrapper/maven-wrapper.properties to use Maven 3.9.11 and download from https://repo.maven.apache.org/maven2
and the result is consistent with the log messages.
This happens because only-mvnw explicitly unsets MVNW_REPOURL:
so by the time WrapperMojo tries to read the variable it sees null:
As far as I can see, this bug does not exist in only-mvnw.cmd.
The issue is exacerbated by the user property distributionUrl value representing the "final" URl. That is, it is not simply a domain substitution and therefore cannot take the place of MVNW_REPOURL.
- Ngôn ngữ chính
- Java
- Star
- 253
- Fork
- 77
- Merge trung bình
- 22 giờ 10 phút
- Pull request đã merge (30 ngày)
- 7
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của apache/maven-wrapper
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
apache/maven-wrapper#459 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
mvnw.cmd selects the last matching extracted directory while mvnw selects the firstCó thể đã có người làm @tanvir-ux đã nhận 48 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
apache/maven-wrapper#442 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
priority:trivial
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
apache/maven-wrapper#280 · 1 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 57/100
apache/maven-wrapper#461 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
apache/maven-wrapper#454 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của apache/maven-wrapper
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
apache/rocketmq-dashboard#7250 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
ignatov/intellij-erlang#1093 ·
-
HttpMaxFrameSizeTest.sendOversizedMessageTest() passes silently when maxFrameSize is not enforcedĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
apache/activemq#2706 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
federicoiosue/Omni-Notes#1032 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
unitycatalog/unitycatalog#2025 ·
Maintainer thường phản hồi trong vòng 4 ngày