-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feedback on README-BUILD-NEWER-VERSIONS.md (v0.0.2) #4
Comments
Very good how-to, I got an error at the first cherry pick . https://github.com/btactic-oo/unlimited-onlyoffice-package-builder/blob/v0.0.2/README-BUILD-NEWER-VERSIONS.md#apply-no-limits-to-our-repos-desktopm and had to do git stash. Othewise everything was straightforward. I didn't check this page first though and wonder if I can remove the tags later in the process like just before building or if I need to redo the guide. |
Also, since we know the build can and will go wrong,maybe we should explain from where you should start from scratch. Is it enough to just delete the old docker images and start the build script again or do I need to start from another point in the how-to |
Is it also possible to stop the build before it makes the deb packages for those that don't need a .deb file. Or is better to just unpack the deb file and use that? |
I would like to reproduce your error to see if it's expected or not. |
You need to remove both remove both In any case you have remove them just before the Github Actions's Push to build section. Otherwise the push described failed there ( I will update the original warning so that it's easier to understand. |
I think you should be fine by just removing or renaming the In any case if there's a more deep problem you might want to delete the docker container so that the docker build actually gets built from scratch and not rebuild from cache. TODO: Add an If something goes wrong subsection inside the Build (BUILDM) section. |
If you want to build a Docker image or an RPM you'd better start a new issue and we can discuss it there. But in a nutshell what you want is to add the The original:
gets converted onto:
. |
I was trying 8.1.3.3. I have successfully built according to the how-to I had to restart once because of a error cloning some repo. Since you don't delete the out folder all binaries are in there but it's nice to be able to skip the deb building part so appreciate the --binaries-only. Now to see if I can get it to run on my server :) |
Nice, I don't need to fix the how-to regarding the first cherry-pick then. Or do I? |
It worked!!! Thank you so much, you going to gartner symposium in november @adriangibanelbtactic?
You could add a if you get a error that you can't cherry pick then run git stash? |
:) |
No, I'm not. |
|
After I Stashed I could cherry-pick. I'll try and do everything from scratch and post the exact error if it occurs again.
Darnit, I thought I could get you a thank you gift/Beer 😁 |
README-BUILD-NEWER-VERSIONS.md v0.0.2 is available to anyone.
Now you can easily recreate the btactic-oo builds and report back which versions need further changes (or not).
Feedback is welcome so that we polish the document.
Please notice that if you try to build 8.1.3.3 version thanks to Github Actions you should remove
onlyoffice-unlimited-build-debian-11/8.1.3.3
andbuilds-debian-11/8.1.3.3
tags from your unlimited-onlyoffice-package-builder repo just after you fork it from ours or, in any case, just before the Github Actions's Push to build section.The text was updated successfully, but these errors were encountered: