Bug 274534 - games/minecraft-server: Update to 1.20.2
Summary: games/minecraft-server: Update to 1.20.2
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Vladimir Druzenko
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-17 13:16 UTC by Alexandre Labarre
Modified: 2023-11-24 14:11 UTC (History)
4 users (show)

See Also:
freebsd: maintainer-feedback+


Attachments
Update from 1.20.1 to 1.20.2 (626 bytes, text/plain)
2023-10-17 13:16 UTC, Alexandre Labarre
no flags Details
Update from 1.20.1 to 1.20.2 (1.23 KB, patch)
2023-10-17 13:24 UTC, Alexandre Labarre
no flags Details | Diff
Update from 1.20.1 to 1.20.2 with pkg-plist updated (4.76 KB, patch)
2023-10-19 13:54 UTC, Alexandre Labarre
no flags Details | Diff
Update from 1.20.1 to 1.20.2 with pkg-plist updated (6.67 KB, patch)
2023-10-19 19:35 UTC, Alexandre Labarre
no flags Details | Diff
Update from 1.20.1 to 1.20.2 with pkg-plist updated (6.67 KB, patch)
2023-10-19 21:25 UTC, Alexandre Labarre
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexandre Labarre 2023-10-17 13:16:55 UTC
Created attachment 245694 [details]
Update from 1.20.1 to 1.20.2

This patch updates games/minecraft-server to 1.20.2

The only changes are the version number, the master site URL and the distinfo.
Master URL found on official website https://www.minecraft.net/en-us/download/server

Tested on FreeBSD 13.2-RELEASE AMD64.
Comment 1 Alexandre Labarre 2023-10-17 13:24:57 UTC
Created attachment 245695 [details]
Update from 1.20.1 to 1.20.2

distinfo was missing from previous patch. I corrected this
Comment 2 Vladimir Druzenko freebsd_committer freebsd_triage 2023-10-17 20:32:29 UTC
Did you check minecraft-server/pkg-plist?
@rmtry part.
Check patch for previous update for more information.
Comment 3 Alexandre Labarre 2023-10-19 13:54:27 UTC
Created attachment 245742 [details]
Update from 1.20.1 to 1.20.2 with pkg-plist updated

Update from 1.20.1 to 1.20.2 with pkg-plist updated

I added foolowing files with @rmtry in pkg-plist. This list appears when starting application the first time.

Unpacking 1.20.2/server-1.20.2.jar (versions:1.20.2) to versions/1.20.2/server-1.20.2.jar
Unpacking com/google/guava/failureaccess/1.0.1/failureaccess-1.0.1.jar (libraries:com.google.guava:failureaccess:1.0.1) to libraries/com/google/guava/failureaccess/1.0.1/failureaccess-1.0.1.jar
Unpacking com/mojang/brigadier/1.1.8/brigadier-1.1.8.jar (libraries:com.mojang:brigadier:1.1.8) to libraries/com/mojang/brigadier/1.1.8/brigadier-1.1.8.jar
Unpacking com/mojang/datafixerupper/6.0.8/datafixerupper-6.0.8.jar (libraries:com.mojang:datafixerupper:6.0.8) to libraries/com/mojang/datafixerupper/6.0.8/datafixerupper-6.0.8.jar
Unpacking com/mojang/logging/1.1.1/logging-1.1.1.jar (libraries:com.mojang:logging:1.1.1) to libraries/com/mojang/logging/1.1.1/logging-1.1.1.jar
Unpacking net/sf/jopt-simple/jopt-simple/5.0.4/jopt-simple-5.0.4.jar (libraries:net.sf.jopt-simple:jopt-simple:5.0.4) to libraries/net/sf/jopt-simple/jopt-simple/5.0.4/jopt-simple-5.0.4.jar
Unpacking org/apache/logging/log4j/log4j-api/2.19.0/log4j-api-2.19.0.jar (libraries:org.apache.logging.log4j:log4j-api:2.19.0) to libraries/org/apache/logging/log4j/log4j-api/2.19.0/log4j-api-2.19.0.jar
Unpacking org/apache/logging/log4j/log4j-core/2.19.0/log4j-core-2.19.0.jar (libraries:org.apache.logging.log4j:log4j-core:2.19.0) to libraries/org/apache/logging/log4j/log4j-core/2.19.0/log4j-core-2.19.0.jar
Unpacking org/apache/logging/log4j/log4j-slf4j2-impl/2.19.0/log4j-slf4j2-impl-2.19.0.jar (libraries:org.apache.logging.log4j:log4j-slf4j2-impl:2.19.0) to libraries/org/apache/logging/log4j/log4j-slf4j2-impl/2.19.0/log4j-slf4j2-impl-2.19.0.jar
Unpacking org/joml/joml/1.10.5/joml-1.10.5.jar (libraries:org.joml:joml:1.10.5) to libraries/org/joml/joml/1.10.5/joml-1.10.5.jar
Comment 4 Vladimir Druzenko freebsd_committer freebsd_triage 2023-10-19 15:52:56 UTC
(In reply to Alexandre Labarre from comment #3)
Other files not appearing?
You removed a lot of lines from pkg-list.
Previous patch I tested on real game and got list of files after start/stop game.
Comment 5 Alexandre Labarre 2023-10-19 19:35:12 UTC
Created attachment 245749 [details]
Update from 1.20.1 to 1.20.2 with pkg-plist updated

(In reply to Vladimir Druzenko from comment #4)
I entirely deleted /usr/local/etc/minecraft-server and /usr/local/minecraft-server where files/libraries are copied during first start, then I started process again.
Now it seems better, and conform with files referenced in your pkg-plist for 1.20.1 version.
Comment 6 Vladimir Druzenko freebsd_committer freebsd_triage 2023-10-19 20:31:14 UTC
(In reply to Alexandre Labarre from comment #5)
Is sorted list line 
@rmtry minecraft-server/versions/1.20.2/server-1.20.2.jar
must be near end of the file.
Keep pkg-plist sorted alphabetical.

Then waiting approve from maintainer and I'll commit.
Comment 7 Alexandre Labarre 2023-10-19 21:25:43 UTC
Created attachment 245750 [details]
Update from 1.20.1 to 1.20.2 with pkg-plist updated

(In reply to Vladimir Druzenko from comment #6)
Thank you for corrected me.
@rmtry entry list is now sorted alphabetically.
Comment 8 Jonathan Price 2023-10-19 21:32:36 UTC
Comment on attachment 245750 [details]
Update from 1.20.1 to 1.20.2 with pkg-plist updated

Thanks, both!
Comment 9 Alexandre Labarre 2023-10-19 22:21:46 UTC
Thank you @Vladimir for your good advices and corrections and thank you @Jonathan for your validation.

I also add the changelog from official Minecraft website:
https://www.minecraft.net/en-us/article/minecraft-java-edition-1-20-2
Comment 10 Vladimir Druzenko freebsd_committer freebsd_triage 2023-10-22 09:10:31 UTC
Question to maintainer: why do you use a lot of the "@rmtry minecraft-server/libraries/*" instead of the "@postunexec rm -rf %D/minecraft-server/libraries"?
Comment 12 commit-hook freebsd_committer freebsd_triage 2023-11-24 14:10:42 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=62a61b0aa8e01598c56d609d76eee107a3cabdab

commit 62a61b0aa8e01598c56d609d76eee107a3cabdab
Author:     Alexandre Labarre <alexandre.labarre@gmail.com>
AuthorDate: 2023-11-24 14:03:48 +0000
Commit:     Vladimir Druzenko <vvd@FreeBSD.org>
CommitDate: 2023-11-24 14:09:19 +0000

    games/minecraft-server: update to 1.20.2

    Changelog:
    https://www.minecraft.net/en-us/article/minecraft-java-edition-1-20-2

    PR:             274534
    Approved by:    Jonathan Price <freebsd@jonathanprice.org> (maintainer), tcberner (mentor), arrowd (mentor)

 games/minecraft-server/Makefile  |  4 ++--
 games/minecraft-server/distinfo  |  6 +++---
 games/minecraft-server/pkg-plist | 42 ++++++++++++++++++++--------------------
 3 files changed, 26 insertions(+), 26 deletions(-)
Comment 13 Vladimir Druzenko freebsd_committer freebsd_triage 2023-11-24 14:11:21 UTC
Sorry for delay. Commited.
Thanks!