Smiley-McSmiles / jellyman

Jellyman is a set of scripts to install/manage and update the jellyfin-combined tar.gz generic linux package
GNU General Public License v3.0
92 stars 4 forks source link

The script is not working for latest build version jellyfin 10.8.13 #54

Closed hobby-lc closed 9 months ago

hobby-lc commented 9 months ago

The filename of the latest build version is "jellyfin_10.8.13_amd64.tar.gz". The regex used in setup.sh line 258 is not able to process '.13' in the file name. FYI.

jellyfin_archive=$(echo "$jellyfinIndex" | grep -o jellyfin_[0-9][0-9].[0-9].[0-9].$architecture.tar.gz | head -1)

https://github.com/Smiley-McSmiles/jellyman/blob/14711dbc07f0be3b867532ffc9e35d81dd50913b/setup.sh#L258C85-L258C85`

Smiley-McSmiles commented 9 months ago

I'll look into submitting a patch tomorrow. Thank you for the digging here.

Smiley-McSmiles commented 9 months ago

I submitted the newest release v1.6.9 to fix this issue. Thank you again for bringing this to my attention.

hobby-lc commented 9 months ago

Great thanks for the update.

function Download_version() may also suffer from same the issue such that there are some missing characters in the variable versionList when running jellyman -vd:

https://github.com/Smiley-McSmiles/jellyman/blob/df133dbe4a903d7f373e42d173b7d24e38f1a2db/scripts/jellyman#L128

Current Jellyfin version installed
jellyfin_10.8.13

Please select a stable version:

     1  10.8.0
     2  10.8.1
     3  10.8.1
     4  10.8.1
     5  10.8.1
     6  10.8.1
     7  10.8.2
     8  10.8.3
     9  10.8.4
    10  10.8.5
    11  10.8.6
    12  10.8.7
    13  10.8.8
    14  10.8.9
Please enter the number corresponding with
the version you want to install [1-14] :

Correct labels for items 3,4,5,6 should be

     3  10.8.10
     4  10.8.11
     5  10.8.12
     6  10.8.13