Git Product home page Git Product logo

Comments (6)

Etienne13 avatar Etienne13 commented on September 17, 2024

Fixed the issue by adapting my code generator to define pok_ports_names_max_len variable.

from pok.

sduverger avatar sduverger commented on September 17, 2024

Hi Etienne,

Julien is right, there is a fix in my patch for #9 with an AWK one-liner to generate pok_port_names_max_len during compilation phase. Then this variable is extern in syscall.c and should be conditionally included only if we NEED_PORT.

I didn't touch your perl script and i think it might be the correct way to generate this variable, but if you do so, don't forget to remove my one-liner in the Makefile.

from pok.

sduverger avatar sduverger commented on September 17, 2024

Did you commit anything ? i don't see your changes.

from pok.

Etienne13 avatar Etienne13 commented on September 17, 2024

The fix was in my AADL to pok code generator (on another repository).

I saw the AWK part but it seems the problem was in my tool and did not appear in previous versions for a reason I do not understand. I also had a problem in conflicting definitions of strlen but I fixed it in my latest commit: it appeared yesterday and did not show up before.

Note/question: if the variable is defined in deployment.c, wouldn't the awk part lead to a redefinition of the variable after it is copied in sizes.c?

from pok.

sduverger avatar sduverger commented on September 17, 2024

I generated it in "sizes.c" as it is related to size questions :) If you also generate it in “deployment.c" obviously there will be a conflict.

Whatever your AADL generated does, this should not impact the compilation phase of the "generated" code, as the one-liner is called during final compilation phase.

I think you should take the decision on your own, whether or not doing it in the Makefile or in the perl configuration script (which i think is the best option as it is responsible for port names generation too).

By the way i didn't see the "#ifdef" around the "extern" definition of the "pok_port_names_max_len" in "syscall.c". Did you commit it ? It is mandatory, because if we do not make uses of POK ports in an example, there will be no definition of the variable outside "syscall.c".

from pok.

Etienne13 avatar Etienne13 commented on September 17, 2024

I just did the push

from pok.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.