This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC] Handle GPC specific name for main function


> >   /* The name of the symbol that GPC uses as the name of the main
> >      subprogram (since version ...).  */
> 
>   Here, I really need feedback from GPC developers!

This makes sense. I thought you had already consulted with the GPC
developers...

> > You also need to update Makefile.in, since you added a dependency.
> 
>   Funny, while trying to add this, I noticed that
> ${ada_lang_h} was twice in the dependency list, so
> I just replaced the second one with ${p_lang_h}

Excellent, thanks for catching and fixing this.

> > With these adjustments in, your patch is pre-approved (but please
> > remember to give it a round of testing, just in cases).
> 
>   Thanks, but my problem is that I am unable to run the testsuite,
> and anyhow gdb.pascal is not yet in CVS,
> and its mainly there where I expect a change.

I am mostly concerned about the effect on the other languages.
I just want to make sure that the rest of the testsuite results
stay unchanged. If you don't have access to a machine where you
can run the testsuite, then let me know, and I'll give it a quick
run on one of our fast machines for you. Your change looks relatively
straightforward, but it never hurts to be cautious.

-- 
Joel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]