+
Skip to content

Conversation

bleykauf
Copy link

@bleykauf bleykauf commented Jul 5, 2023

No description provided.

@bleykauf bleykauf merged commit 56926b1 into master Jul 5, 2023
Copy link

@doronbehar doronbehar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @bleykauf . I'm working on some improvements related to deployment topics we discussed in the past. I'm trying to adapt to linien 0.8.0 and I encountered this issue.

python_requires=">=3.5",
install_requires=[
"myhdl>=0.11",
"rpyc>=4.0,<5.0",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought that rpyc >5.0 compatibility was fixed in version 0.3.2 of linien (according to linien's CHANGELOG.md).

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The problem with rpyc>=5.0 was that it was incompatible with the outdated python version of RedPitaya OS 1.0. The fix in that version is that the installed version was explicitly set to <5.0 to avoid conflicts between server and client. RedPitaya OS 2.0 apparently now has a stable release and the next version of Linien will only support OS 2.0.

That means that the requirement will be rpyc>=5.0,<6.0 from then on.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The problem with rpyc>=5.0 was that it was incompatible with the outdated python version of RedPitaya OS 1.0

Python supports constraining that with the python version, as explained here: https://stackoverflow.com/questions/21082091/install-requires-based-on-python-version

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But we have to make sure that we always use the same version of rpyc for server and client since they are incompatible. Will be upgraded to >=5.0,<6.0 with the next release.

@bleykauf bleykauf deleted the release/1.2.0 branch December 15, 2023 11:37
doronbehar added a commit to doronbehar/nixpkgs that referenced this pull request Jan 24, 2024
This version is required for linien-common, due to RedPitaya OS Python
version limitation. See:
linien-org/pyrp3#10 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载