erpc icon indicating copy to clipboard operation
erpc copied to clipboard

remove defining parse.error which make errors while compiling erpcgen…

Open amgross opened this issue 3 years ago • 2 comments

remove defining parse.error which make errors while compiling erpcgen.exe

Pull request

Choose Correct

  • [X] bug
  • [ ] feature

Describe the pull request

see issue #314

To Reproduce

Expected behavior

Screenshots

Desktop (please complete the following information):

  • OS:Windows
  • eRPC Version:1.8.1

Steps you didn't forgot to do

  • [X] I checked if other PR isn't solving this issue.
  • [x] I read Contribution details and did appropriate actions.
  • [X] PR code is tested. In windows system only
  • [x] PR code is formatted.
  • [x] Allow edits from maintainers pull request option is set (recommended).

Additional context

amgross avatar Oct 02 '22 05:10 amgross

Hi @amgross , i would like to keep this and found workround for you. Could you post more information? Bison version and is it from this site? https://sourceforge.net/projects/winflexbison Maybe we need update that line to something else...

Hadatko avatar Oct 02 '22 15:10 Hadatko

hi @Hadatko .It is the one available now at https://sourceforge.net/projects/winflexbison I suppose it is latest

amgross avatar Oct 06 '22 07:10 amgross

@amgross your version looks old if you download the on that page directly. Could you try use version3 from same link under files? https://sourceforge.net/projects/winflexbison/files/win_flex_bison3-latest.zip/download

Hadatko avatar Dec 14 '22 13:12 Hadatko

Will take me few days to get to it, but I will try. I will update

amgross avatar Dec 14 '22 18:12 amgross

Thank you and let us know. In this case we will just update Readme. Otherwise we will need to fix it somehow differently. But the issue you are mentioning looks be connected to bison version.

Hadatko avatar Dec 14 '22 18:12 Hadatko

With it getting 'configuration 'Debug|Win32'. IntelliSense might be unavailable.' error.

Whatever.

amgross avatar Dec 19 '22 08:12 amgross