papampi / nvOC_miners

nvOC easy-to-use Linux Nvidia Mining OS Miners
8 stars 7 forks source link

Added new miners and updated some #4

Closed papampi closed 6 years ago

papampi commented 6 years ago

Added some miners and edited miner update script in 19-2.1 branch. Forgot to make new branch and pull request so you can check easier.

Please when you get time, take a look and see if all is ok. Thanks.

LuKePicci commented 6 years ago

Almost good, the only thing you have to fdo is remove all "version" file from the repo folders, they have to be only inside the binaries archive otherwise the version checking is broken. Meanwhile would be better to use xz compression instead of gz, it's way more efficient.

Alysko commented 6 years ago

Why not to add wget to each miner repo instead of making tar and include them in git?

LuKePicci commented 6 years ago

Because each miner may provide a different kind of tarball, and there's no reason to depend on them. We only need the single executable binary, and most of them does not provide such kind of a tarball. Moreover, there is not a good point in the updating workflow where to perform the actual download. If you pull the updated git miners submodule you would then execute a further download script, before installing.

LuKePicci commented 6 years ago

And of course you would loose the ability to revert/switch to different branches of miners without having to implement such functionality with a smart combination of wget commands and old binary move/rename

Alysko commented 6 years ago

OK. Line 166: cat ${NVOC_MINERS}/bminer/bminer-v8.0.0.tar.gz -> tar.xz

LuKePicci commented 6 years ago

Yup, there are a few other also.

LuKePicci commented 6 years ago

@papampi , because of a git limitation in submodule handling, shallow clones of submodules tracking a specific branch which is not the primary one does not work. We have to take one of the following alternative actions:

It depends on how you used to manage the legacy update script, if you need the master branch continue working for legacy support I would opt for the last one.

Note: a shallow clone is what allows not to fetch the whole repo with binary history of all branches

papampi commented 6 years ago

I think better to keep the master until the stable release of 19-2.1, then we merge 19-2.1 to master Users may still want to install 19-2.0 and need to update miners from master repo. We can set 19-2.1 as primary for now

LuKePicci commented 6 years ago

Ok, let me know whan you did it, I will update the wiki as soon as I test it successfully.

papampi commented 6 years ago

19-2.1 is base branch

LuKePicci commented 6 years ago

Fine, now it works as expected, it only downloads latest versions while cloning with commands I previously wrote in the wiki page.