search

madslundt / cloud-media-scripts

Upload and stream media from the cloud with or without encryption. Cache all new and recently streamed media locally to access quickly and reduce API calls
MIT License
89 stars 11 forks source link

Added mergerfs/unionfs option for deletion ability. #44

Closed Dulanic closed 6 years ago

Dulanic commented 6 years ago

Set pool_choice to change the ability to use mergerfs vs unionfs. Tested and both options are working. Also moved options up in config to top to consolidate and make easier to read.

image

Dulanic commented 6 years ago

And more fully tested so this wont be an issue :) default will be /w unionfs /w options for mergerfs.

madslundt commented 6 years ago

That is really awesome. Thank you.

Dulanic commented 6 years ago

Will do

Dulanic commented 6 years ago

I can adjust I used the compile version because then don't have to worry about varying distributions. What error did u see? I also corrected an error in my original code that was always compiling. That's in next PR.

On Mon, Mar 5, 2018, 11:33 AM Antexa notifications@github.com wrote:

@Antexa commented on this pull request.

In setup.sh https://github.com/madslundt/cloud-media-scripts/pull/44#discussion_r172245102 :

@@ -49,6 +49,18 @@ if [ ! -d "${rclone_dir}" ]; then mkdir -p "${rclone_dir}" fi

+if [ "$(pool_choice)" = "1" ]; then

  • if [ -f "${mfs_bin}" ]; then
  • git clone https://github.com/trapexit/mergerfs.git
  • cd mergerfs
  • sudo make install-build-pkgs

  • build-essential git g++ debhelper libattr1-dev python automake libtool lsb-release

  • make deb
  • sudo dpkg -i ../mergerfs_version_arch.deb

I got several errors with this code. Why not use precompiled packages?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/madslundt/cloud-media-scripts/pull/44#pullrequestreview-101229477, or mute the thread https://github.com/notifications/unsubscribe-auth/ACz8buPZBAlaOnzMOkGBh6NgJNS6V1Rxks5tbXb3gaJpZM4SbpnI .

Antexa commented 6 years ago

Nevermind, it seems to be correct now... I may have missed something the first time around.

Dulanic commented 6 years ago

OK, if you see a problem let me know for sure. May need to ensure the dependencies exist too.... I will look into that. Until @madslundt get's a chance to look over my PR https://github.com/madslundt/cloud-media-scripts/pull/45 the script is slightly off that it tried to build the package if you do have it, not if your don't. One day I'll be better at ensuring I don't break something lol

apt-get install g++ pkg-config git git-buildpackage pandoc debhelper libfuse-dev libattr1-dev -y