0001-Initial-Commit.patch3 KBDownload
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | JoshStrobl | T1023 rclone - Command line program to sync files between popular cloud services | |||
| Resolved | JoshStrobl | T1039 rclone .patch |
Event Timeline
Comment Actions
- Needs to use their release tarballs, not git commits.
- Don't use network.base, that means it'll be installed onto everyone's machines. Please use network.util.
- Any particular reason why a faux workspace is being created rather than just setting the workdir as the Go path?
- # Have to set the GOPATH again, I don't knwo why,but it has probably something to do how eopkg works. can be removed. It isn't anything to do with eopkg, it's ypkg and evobuild stuff. Exporting vars on individual steps is normal.
- Drop the first two redundant install commands and read what -D does :) Prepend them to the flags on the binary and manual file installation.
Comment Actions
- I already talked about this with the Author, i guess there will be a new release soon.
- Okay, changed it.
- You are right this is a loot simpler
- Okay, thanks didn't knew this
- Didn't quit get what you mean with "Prepend them to the flags on the binary and manual file installation."
I'll update this patch as soon as there is a new release.
Comment Actions
I already talked about this with the Author, i guess there will be a new release soon.
What? They have a releases section, with tarballs. Click the Releases link on https://github.com/ncw/rclone and you'll see the versioned tarballs.
Comment Actions
Yes, i know, the problem is that the latest release is broken, that is why i used a commit as source.
Comment Actions
Yes, i know, the problem is that the latest release is broken, that is why i used a commit as source.
The more accepted practice is to use latest tarball then apply patches based on git commits that resolve said issue (or to indeed wait for next release).
Comment Actions
Updated the .patch
Not sure if I made the .patch for building right, but it works, maybe someone can have a look on it.
Comment Actions
Yea that patch doesn't look right at all. Should really only include the files that changed, not what looks like every single file between the latest version and master.
Comment Actions
So I should only make the git diff between the release and the commit with the fix?
I downloaded the realease 1.33 and the master.
And than I used this command "git diff rclone-1.33/ rclone/"
Comment Actions
I think we should just wait a little longer for the next release rather than patching in all of master. I'm still unsure why you couldn't just do the 1.33 release for now.