Skip to content

Conversation

elupus
Copy link

@elupus elupus commented Jun 23, 2025

  • Add support for building p-net from the evaluation binaries and samples provided on our releases.
  • Rework src based build to build from a locally provided src file instead of github repository.

This allow us to build p-net from releases instead
of source code which i useful for evaluation.
@elupus elupus force-pushed the rtljopl/binary_build branch from e2c610b to 19331dc Compare June 24, 2025 08:44
@elupus elupus requested review from pyhys and amn-rtl June 24, 2025 11:51
@elupus elupus marked this pull request as ready for review June 24, 2025 11:51
@elupus elupus force-pushed the rtljopl/binary_build branch from 6500216 to 419d7f2 Compare June 27, 2025 06:03
Copy link

@amn-rtl amn-rtl left a comment

Choose a reason for hiding this comment

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

Some comments

@@ -0,0 +1,4 @@
# P-Net Yocto Files

Place source code of p-net in `p-net-sources.zip` here
Copy link

Choose a reason for hiding this comment

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

Update the zip file name to correspond to the recipe.

LIC_FILES_CHKSUM = "file://CMakeLists.txt;beginline=1;endline=14;md5=aa7e6f963fad674b02c35dc8e9aa5519"
SECTION = "console/network"

DEPENDS = "p-net-eval"
Copy link

Choose a reason for hiding this comment

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

Where is "p-net-eval" defined?


install -d ${D}${prefix}/share/profinet
install -m 0755 ${S}/share/profinet/* ${D}${prefix}/share/profinet
}
Copy link

Choose a reason for hiding this comment

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

Is the intention to install a static library in the image? If so, does this actually work?
(My experience is that a "-staticdev" package is necessary to install a static library.)

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.

4 participants