-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
ffsync: Update syncstorage-rs to v0.17.3 #6187
Conversation
hey @hgy59, I'm not very familiar with rust packages and was wondering if you could assist with troubleshooting this update. Thus far the builds seem to be failing with this message:
After doing some research it seems to be similar to this issue: diesel-rs/diesel#4056 but the solutions presented don't seem to resolve the issue with the build. |
@mreid-tt https://github.com/sgrif/mysqlclient-sys does not provide an official release 0.4.0, latest is still 0.3.0. Probably it is too early to use 0.4.0. |
we have
There are probably other settings that must be defined in the environment. unfortunately I can't see the real build error... |
Understood. In the meantime I've tried an alternate approach suggested in diesel-rs/diesel#4056. Rather than using the latest diesel v2.2.2, I used v2.1.6. This allowed the builds for
Further up in the log file I see:
Perhaps you can help with the above error? |
@mreid-tt @th0ma7 The first errror when building python is:
probably something in the python wheel creation has changed (update of pip, setuptools, ...?) Found similar error here: |
The build of python311 fails for several archs for arch-armv7-7.1 the following error occurred while creating the cross environment:
My local build of python311 does not fail for arch-armv7-7.1. my successful build used a 12 months old ghcr.io/synocommunity/spksrc with sha 65b3e95ec808 |
you can ignore errors in python tests. Most of the tests fail for cross compiled targets. The tests are executed to optimize the python code. So optimization for cross compiled versions is limitted. |
Includes update to diesel 2.2.2
This reverts commit 814696a.
13612a0
to
385abfb
Compare
@hgy59, it seems that the latest changes to the repo fixed the build issues. A re-basing allowed my PR to build without issue. |
This reverts commit 88e157d.
it was at least worth the effort... |
@hgy59 I think I need some help with the requirements files. Comparing the two releases (mozilla-services/syncstorage-rs@0.14.3...0.17.3), and looking at the requirements files I see the following changes: requirements.txt
tools/tokenserver/requirements.txt
I'm not sure how to incorporate these changes into the EDIT: I looked back at your recommendations for this in #5939 (comment) but as I don't have a local dev environment (no x86 arch), I don't think your script to generate the requirements will work for me. EDIT: I've looked back at additional recommendations from @th0ma7 in #5939 (comment) and I've made a go at updating the requirements. Hopefully it will build okay. EDIT: The build seems to have been successful and I've tested an upgrade install which works well. I updated the named dependency ( |
There must be an error somewhere
|
Hmm, looking at the logs there are a number of warnings about statements falling through and deprecations but I couldn't see anything specific in the
This happens during the installation of multiple versions of |
It's not clear to me what is running at the time this error is thrown. The Outside of that the package installs correctly and I've configured Firefox clients to point to the server that is installed and the SyncStorage service works perfectly. I really can't detect any impact from this error. |
Don't worry about this one in particular as it is when i am forcing a downgrade of pip to install an older numpy in the crossenv. I then re-upgrade it to latest with no issue |
@hgy59 I've resolved the build error as the log now looks like this:
When I looked more into the build sequence the last command before the error was in this section: Line 340 in 2dc9159
The next sections after this dealt with "service configuration for admin port" as well as "DSM UI configuration (app/config)". Part of this dealt with the service description. I guessed that for the syntax error shown it must have something to do with quotes. I noted that the description of the package had three single quotes in the description: Line 17 in 2dc9159
I then did a search in the repo for other packages with single quotes and noted a package that had the single quotes escaped and recently built without error: spksrc/spk/chromaprint/Makefile Line 11 in 2dc9159
So I then simply escaped all the single quotes and the build completed without error. One interesting finding was that the Line 460 in 2dc9159
Either way, it seems safer to escape single quotes. |
I have now updated the description and examples of the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ready to merge and publish (if @th0ma7 has no objections regarding python)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, and have'nt found any changes to the pyhton311 neither, have i missed anything @hgy59 ?
Description
This PR includes the following:
Fixes #
Checklist
all-supported
completed successfullyType of change