Skip to content
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

Request for latest release #103

Open
Shaikh-Ubaid opened this issue Jul 6, 2023 · 12 comments
Open

Request for latest release #103

Shaikh-Ubaid opened this issue Jul 6, 2023 · 12 comments

Comments

@Shaikh-Ubaid
Copy link

Shaikh-Ubaid commented Jul 6, 2023

I am facing this issue #89 using the latest available bision 3.8.2. It seems it is fixed in the current master branch, but is not yet available with the release version. The most recent release available is from September 2021. I would like to request if you could make a new release. Thank you for your time and consideration.

@torokati44
Copy link

+1

1 similar comment
@0O0o0oOoO00
Copy link

+1

@torokati44
Copy link

bump

@Shaikh-Ubaid
Copy link
Author

For now, we post-process the generated parser.tab.cc file by adding [[maybe_unused]] to the declaration of yynerrs. It is a short 4 line code, available at https://github.com/lcompilers/lpython/blob/3ed309840da9ae0fbafc797554605bfda6505da4/build0.sh#L21-L24.

@torokati44
Copy link

bump again

@ing-eoking
Copy link

+1

@torokati44
Copy link

+1...?

@cooljeanius
Copy link

I am facing this issue #89 using the latest available bision 3.8.2. It seems it is fixed in the current master branch, but is not yet available with the release version. The most recent release available is from September 2021. I would like to request if you could make a new release. Thank you for your time and consideration.

Yeah same issue, it led to mdbtools/mdbtools#425...

@r-barnes
Copy link

+1

@r-barnes
Copy link

I emailed [email protected] on 27 June, but have not yet received a response.

@cooljeanius
Copy link

I emailed [email protected] on 27 June, but have not yet received a response.

Ah yes, so I see: https://lists.gnu.org/archive/html/bug-bison/2024-06/msg00000.html

Also, more recently than that, I see @ArsenArsen submitted a patch to bison-patches that hasn't received a response yet, either: https://lists.gnu.org/archive/html/bison-patches/2024-07/msg00000.html

@ArsenArsen
Copy link

might be worthwhile to email [email protected] with an inquiry..

bison is currently marked as looking for co-maintainers. now, I don't know enough parsing theory to step up (plus I also have lots of other commitments), but I invite folk to take up that offer and email the address above

dburgener added a commit to dburgener/selint that referenced this issue Oct 23, 2024
Bison in 3.8.2 added new warnings to catch m4 strings accidentally included in the generated code output.  It had the unfortunate side effect of flagging grammars that include "m4_foo" style strings as identifiers in their grammar (which we do).  Based on the thread, it looks like a fix was implemented on the Bison side, but as of three years later, no release has been made.  Bison seems to possibly have maintainer inactivity: akimd/bison#103.

Since a fix for the false positive is not forthcoming, disable the
warning.  Unfortunately, I don't think a more granular disablement is
available, but IMO false positive warnings are worse than false
negatives.
dburgener added a commit to SELinuxProject/selint that referenced this issue Oct 28, 2024
Bison in 3.8.2 added new warnings to catch m4 strings accidentally included in the generated code output.  It had the unfortunate side effect of flagging grammars that include "m4_foo" style strings as identifiers in their grammar (which we do).  Based on the thread, it looks like a fix was implemented on the Bison side, but as of three years later, no release has been made.  Bison seems to possibly have maintainer inactivity: akimd/bison#103.

Since a fix for the false positive is not forthcoming, disable the
warning.  Unfortunately, I don't think a more granular disablement is
available, but IMO false positive warnings are worse than false
negatives.
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

No branches or pull requests

7 participants