Forum: Ruby-core [ruby-trunk - Bug #6789][Open] parse.y compilation error due not updated id.h

Posted by Luis Lavena (luislavena)
on 2012-07-25 00:12
(Received via mailing list)
Issue #6789 has been reported by luislavena (Luis Lavena).

----------------------------------------
Bug #6789: parse.y compilation error due not updated id.h
https://bugs.ruby-lang.org/issues/6789

Author: luislavena (Luis Lavena)
Status: Open
Priority: Normal
Assignee: tenderlovemaking (Aaron Patterson)
Category: build
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-07-25 trunk 36527) [x64-mingw32]


=begin
Hello,

After r36524, compilation with existing build tree will fail due id.h 
not being regenerated:

 compiling ../dmyversion.c
 generating parse.c
 parse.h updated
 compiling ../array.c
 ...
 compiling parse.c
 In file included from ../parse.y:27:0:
 id.h:165:1: error: size of array 'checking_for_idNULL' is negative
 id.h:166:1: error: size of array 'checking_for_idRespond_to' is 
negative
 id.h:167:1: error: size of array 'checking_for_idIFUNC' is negative
 id.h:168:1: error: size of array 'checking_for_idCFUNC' is negative
 id.h:169:1: error: size of array 
'checking_for_id_core_set_method_alias' is negative
 id.h:170:1: error: size of array 
'checking_for_id_core_set_variable_alias' is negative
 id.h:171:1: error: size of array 'checking_for_id_core_undef_method' is 
negative
 id.h:172:1: error: size of array 'checking_for_id_core_define_method' 
is negative
 id.h:173:1: error: size of array 
'checking_for_id_core_define_singleton_method' is negative
 id.h:174:1: error: size of array 'checking_for_id_core_set_postexe' is 
negative
 id.h:175:1: error: size of array 'checking_for_id_core_hash_from_ary' 
is negative
 id.h:176:1: error: size of array 'checking_for_id_core_hash_merge_ary' 
is negative
 id.h:177:1: error: size of array 'checking_for_id_core_hash_merge_ptr' 
is negative
 id.h:178:1: error: size of array 'checking_for_id_core_hash_merge_kwd' 
is negative
 id.h:179:1: error: size of array 'checking_for_tLAST_TOKEN' is negative
 make: *** [parse.o] Error 1

It required a forced removal of id.h to properly compile after.

This caused a build failure when building RubyInstaller from existing 
compilation:

http://ci.rubyinstaller.org/job/ruby-trunk-x64/44/console
http://ci.rubyinstaller.org/job/ruby-trunk-x86/45/console

Perhaps id.h generation needs to be marked as dependency of parse.c 
itself?

=end
Posted by Luis Lavena (luislavena)
on 2012-07-25 00:13
(Received via mailing list)
Issue #6789 has been updated by luislavena (Luis Lavena).

Status changed from Open to Assigned


----------------------------------------
Bug #6789: parse.y compilation error due not updated id.h
https://bugs.ruby-lang.org/issues/6789#change-28412

Author: luislavena (Luis Lavena)
Status: Assigned
Priority: Normal
Assignee: tenderlovemaking (Aaron Patterson)
Category: build
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-07-25 trunk 36527) [x64-mingw32]


=begin
Hello,

After r36524, compilation with existing build tree will fail due id.h 
not being regenerated:

 compiling ../dmyversion.c
 generating parse.c
 parse.h updated
 compiling ../array.c
 ...
 compiling parse.c
 In file included from ../parse.y:27:0:
 id.h:165:1: error: size of array 'checking_for_idNULL' is negative
 id.h:166:1: error: size of array 'checking_for_idRespond_to' is 
negative
 id.h:167:1: error: size of array 'checking_for_idIFUNC' is negative
 id.h:168:1: error: size of array 'checking_for_idCFUNC' is negative
 id.h:169:1: error: size of array 
'checking_for_id_core_set_method_alias' is negative
 id.h:170:1: error: size of array 
'checking_for_id_core_set_variable_alias' is negative
 id.h:171:1: error: size of array 'checking_for_id_core_undef_method' is 
negative
 id.h:172:1: error: size of array 'checking_for_id_core_define_method' 
is negative
 id.h:173:1: error: size of array 
'checking_for_id_core_define_singleton_method' is negative
 id.h:174:1: error: size of array 'checking_for_id_core_set_postexe' is 
negative
 id.h:175:1: error: size of array 'checking_for_id_core_hash_from_ary' 
is negative
 id.h:176:1: error: size of array 'checking_for_id_core_hash_merge_ary' 
is negative
 id.h:177:1: error: size of array 'checking_for_id_core_hash_merge_ptr' 
is negative
 id.h:178:1: error: size of array 'checking_for_id_core_hash_merge_kwd' 
is negative
 id.h:179:1: error: size of array 'checking_for_tLAST_TOKEN' is negative
 make: *** [parse.o] Error 1

It required a forced removal of id.h to properly compile after.

This caused a build failure when building RubyInstaller from existing 
compilation:

http://ci.rubyinstaller.org/job/ruby-trunk-x64/44/console
http://ci.rubyinstaller.org/job/ruby-trunk-x86/45/console

Perhaps id.h generation needs to be marked as dependency of parse.c 
itself?

=end
Posted by Aaron Patterson (tenderlove)
on 2012-07-26 01:46
(Received via mailing list)
On Wed, Jul 25, 2012 at 07:13:27AM +0900, luislavena (Luis Lavena) 
wrote:
> It required a forced removal of id.h to properly compile after.

Doesn't `make clean` take care of this?

> This caused a build failure when building RubyInstaller from existing 
compilation:
>
> http://ci.rubyinstaller.org/job/ruby-trunk-x64/44/console
> http://ci.rubyinstaller.org/job/ruby-trunk-x86/45/console
>
> Perhaps id.h generation needs to be marked as dependency of parse.c itself?

Urgh.  If so, I think this should be assigned to Nobu.  I am extremely
terrible at our autotools / Makefile build system.
Posted by Luis Lavena (luislavena)
on 2012-07-26 01:57
(Received via mailing list)
On Wed, Jul 25, 2012 at 8:45 PM, Aaron Patterson
<tenderlove@ruby-lang.org> wrote:
> On Wed, Jul 25, 2012 at 07:13:27AM +0900, luislavena (Luis Lavena) wrote:
>> It required a forced removal of id.h to properly compile after.
>
> Doesn't `make clean` take care of this?
>

Yes, it does, but the current CI setup is not doing clean before "all
test", will change that.

But others might be building in-tree and not doing clean before
building, expecting dependencies get resolved and files properly
generated.

>
> Urgh.  If so, I think this should be assigned to Nobu.  I am extremely
> terrible at our autotools / Makefile build system.
>

Will reassign, thank you.
Posted by Luis Lavena (luislavena)
on 2012-07-26 01:59
(Received via mailing list)
Issue #6789 has been updated by luislavena (Luis Lavena).

Assignee changed from tenderlovemaking (Aaron Patterson) to nobu 
(Nobuyoshi Nakada)


----------------------------------------
Bug #6789: parse.y compilation error due not updated id.h
https://bugs.ruby-lang.org/issues/6789#change-28448

Author: luislavena (Luis Lavena)
Status: Assigned
Priority: Normal
Assignee: nobu (Nobuyoshi Nakada)
Category: build
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-07-25 trunk 36527) [x64-mingw32]


=begin
Hello,

After r36524, compilation with existing build tree will fail due id.h 
not being regenerated:

 compiling ../dmyversion.c
 generating parse.c
 parse.h updated
 compiling ../array.c
 ...
 compiling parse.c
 In file included from ../parse.y:27:0:
 id.h:165:1: error: size of array 'checking_for_idNULL' is negative
 id.h:166:1: error: size of array 'checking_for_idRespond_to' is 
negative
 id.h:167:1: error: size of array 'checking_for_idIFUNC' is negative
 id.h:168:1: error: size of array 'checking_for_idCFUNC' is negative
 id.h:169:1: error: size of array 
'checking_for_id_core_set_method_alias' is negative
 id.h:170:1: error: size of array 
'checking_for_id_core_set_variable_alias' is negative
 id.h:171:1: error: size of array 'checking_for_id_core_undef_method' is 
negative
 id.h:172:1: error: size of array 'checking_for_id_core_define_method' 
is negative
 id.h:173:1: error: size of array 
'checking_for_id_core_define_singleton_method' is negative
 id.h:174:1: error: size of array 'checking_for_id_core_set_postexe' is 
negative
 id.h:175:1: error: size of array 'checking_for_id_core_hash_from_ary' 
is negative
 id.h:176:1: error: size of array 'checking_for_id_core_hash_merge_ary' 
is negative
 id.h:177:1: error: size of array 'checking_for_id_core_hash_merge_ptr' 
is negative
 id.h:178:1: error: size of array 'checking_for_id_core_hash_merge_kwd' 
is negative
 id.h:179:1: error: size of array 'checking_for_tLAST_TOKEN' is negative
 make: *** [parse.o] Error 1

It required a forced removal of id.h to properly compile after.

This caused a build failure when building RubyInstaller from existing 
compilation:

http://ci.rubyinstaller.org/job/ruby-trunk-x64/44/console
http://ci.rubyinstaller.org/job/ruby-trunk-x86/45/console

Perhaps id.h generation needs to be marked as dependency of parse.c 
itself?

=end
Posted by nagachika (Tomoyuki Chikanaga) (Guest)
on 2012-07-28 19:53
(Received via mailing list)
Issue #6789 has been updated by nagachika (Tomoyuki Chikanaga).

File parse_dependency.patch added

Well, doesn't this patch fix this issue?

----------------------------------------
Bug #6789: parse.y compilation error due not updated id.h
https://bugs.ruby-lang.org/issues/6789#change-28511

Author: luislavena (Luis Lavena)
Status: Assigned
Priority: Normal
Assignee: nobu (Nobuyoshi Nakada)
Category: build
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-07-25 trunk 36527) [x64-mingw32]


=begin
Hello,

After r36524, compilation with existing build tree will fail due id.h 
not being regenerated:

 compiling ../dmyversion.c
 generating parse.c
 parse.h updated
 compiling ../array.c
 ...
 compiling parse.c
 In file included from ../parse.y:27:0:
 id.h:165:1: error: size of array 'checking_for_idNULL' is negative
 id.h:166:1: error: size of array 'checking_for_idRespond_to' is 
negative
 id.h:167:1: error: size of array 'checking_for_idIFUNC' is negative
 id.h:168:1: error: size of array 'checking_for_idCFUNC' is negative
 id.h:169:1: error: size of array 
'checking_for_id_core_set_method_alias' is negative
 id.h:170:1: error: size of array 
'checking_for_id_core_set_variable_alias' is negative
 id.h:171:1: error: size of array 'checking_for_id_core_undef_method' is 
negative
 id.h:172:1: error: size of array 'checking_for_id_core_define_method' 
is negative
 id.h:173:1: error: size of array 
'checking_for_id_core_define_singleton_method' is negative
 id.h:174:1: error: size of array 'checking_for_id_core_set_postexe' is 
negative
 id.h:175:1: error: size of array 'checking_for_id_core_hash_from_ary' 
is negative
 id.h:176:1: error: size of array 'checking_for_id_core_hash_merge_ary' 
is negative
 id.h:177:1: error: size of array 'checking_for_id_core_hash_merge_ptr' 
is negative
 id.h:178:1: error: size of array 'checking_for_id_core_hash_merge_kwd' 
is negative
 id.h:179:1: error: size of array 'checking_for_tLAST_TOKEN' is negative
 make: *** [parse.o] Error 1

It required a forced removal of id.h to properly compile after.

This caused a build failure when building RubyInstaller from existing 
compilation:

http://ci.rubyinstaller.org/job/ruby-trunk-x64/44/console
http://ci.rubyinstaller.org/job/ruby-trunk-x86/45/console

Perhaps id.h generation needs to be marked as dependency of parse.c 
itself?

=end
Posted by Luis Lavena (luislavena)
on 2012-07-28 21:49
(Received via mailing list)
Issue #6789 has been updated by luislavena (Luis Lavena).


nagachika (Tomoyuki Chikanaga) wrote:
> Well, doesn't this patch fix this issue?

Yes, it did.

Can you commit it?

Thank you.
----------------------------------------
Bug #6789: parse.y compilation error due not updated id.h
https://bugs.ruby-lang.org/issues/6789#change-28517

Author: luislavena (Luis Lavena)
Status: Assigned
Priority: Normal
Assignee: nobu (Nobuyoshi Nakada)
Category: build
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-07-25 trunk 36527) [x64-mingw32]


=begin
Hello,

After r36524, compilation with existing build tree will fail due id.h 
not being regenerated:

 compiling ../dmyversion.c
 generating parse.c
 parse.h updated
 compiling ../array.c
 ...
 compiling parse.c
 In file included from ../parse.y:27:0:
 id.h:165:1: error: size of array 'checking_for_idNULL' is negative
 id.h:166:1: error: size of array 'checking_for_idRespond_to' is 
negative
 id.h:167:1: error: size of array 'checking_for_idIFUNC' is negative
 id.h:168:1: error: size of array 'checking_for_idCFUNC' is negative
 id.h:169:1: error: size of array 
'checking_for_id_core_set_method_alias' is negative
 id.h:170:1: error: size of array 
'checking_for_id_core_set_variable_alias' is negative
 id.h:171:1: error: size of array 'checking_for_id_core_undef_method' is 
negative
 id.h:172:1: error: size of array 'checking_for_id_core_define_method' 
is negative
 id.h:173:1: error: size of array 
'checking_for_id_core_define_singleton_method' is negative
 id.h:174:1: error: size of array 'checking_for_id_core_set_postexe' is 
negative
 id.h:175:1: error: size of array 'checking_for_id_core_hash_from_ary' 
is negative
 id.h:176:1: error: size of array 'checking_for_id_core_hash_merge_ary' 
is negative
 id.h:177:1: error: size of array 'checking_for_id_core_hash_merge_ptr' 
is negative
 id.h:178:1: error: size of array 'checking_for_id_core_hash_merge_kwd' 
is negative
 id.h:179:1: error: size of array 'checking_for_tLAST_TOKEN' is negative
 make: *** [parse.o] Error 1

It required a forced removal of id.h to properly compile after.

This caused a build failure when building RubyInstaller from existing 
compilation:

http://ci.rubyinstaller.org/job/ruby-trunk-x64/44/console
http://ci.rubyinstaller.org/job/ruby-trunk-x86/45/console

Perhaps id.h generation needs to be marked as dependency of parse.c 
itself?

=end
Posted by Nobuyoshi Nakada (nobu)
on 2012-08-07 15:33
(Received via mailing list)
Issue #6789 has been updated by nobu (Nobuyoshi Nakada).

Status changed from Closed to Open
% Done changed from 100 to 0


----------------------------------------
Bug #6789: parse.y compilation error due not updated id.h
https://bugs.ruby-lang.org/issues/6789#change-28708

Author: luislavena (Luis Lavena)
Status: Open
Priority: Normal
Assignee: nobu (Nobuyoshi Nakada)
Category: build
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-07-25 trunk 36527) [x64-mingw32]


=begin
Hello,

After r36524, compilation with existing build tree will fail due id.h 
not being regenerated:

 compiling ../dmyversion.c
 generating parse.c
 parse.h updated
 compiling ../array.c
 ...
 compiling parse.c
 In file included from ../parse.y:27:0:
 id.h:165:1: error: size of array 'checking_for_idNULL' is negative
 id.h:166:1: error: size of array 'checking_for_idRespond_to' is 
negative
 id.h:167:1: error: size of array 'checking_for_idIFUNC' is negative
 id.h:168:1: error: size of array 'checking_for_idCFUNC' is negative
 id.h:169:1: error: size of array 
'checking_for_id_core_set_method_alias' is negative
 id.h:170:1: error: size of array 
'checking_for_id_core_set_variable_alias' is negative
 id.h:171:1: error: size of array 'checking_for_id_core_undef_method' is 
negative
 id.h:172:1: error: size of array 'checking_for_id_core_define_method' 
is negative
 id.h:173:1: error: size of array 
'checking_for_id_core_define_singleton_method' is negative
 id.h:174:1: error: size of array 'checking_for_id_core_set_postexe' is 
negative
 id.h:175:1: error: size of array 'checking_for_id_core_hash_from_ary' 
is negative
 id.h:176:1: error: size of array 'checking_for_id_core_hash_merge_ary' 
is negative
 id.h:177:1: error: size of array 'checking_for_id_core_hash_merge_ptr' 
is negative
 id.h:178:1: error: size of array 'checking_for_id_core_hash_merge_kwd' 
is negative
 id.h:179:1: error: size of array 'checking_for_tLAST_TOKEN' is negative
 make: *** [parse.o] Error 1

It required a forced removal of id.h to properly compile after.

This caused a build failure when building RubyInstaller from existing 
compilation:

http://ci.rubyinstaller.org/job/ruby-trunk-x64/44/console
http://ci.rubyinstaller.org/job/ruby-trunk-x86/45/console

Perhaps id.h generation needs to be marked as dependency of parse.c 
itself?

=end
Please log in before posting. Registration is free and takes only a minute.
Existing account (Switch to SSL-encrypted connection)
NEW: Do you have a Google/GoogleMail or Yahoo account? No registration required!
Log in with Google account | Log in with Yahoo account
No account? Register here.