[vsipl++] [patch] install benchmark sources

[Don McCoy]

Jules Bergmann wrote:
> This looks good in principal.  Did you and Stefan work out the issue 
> with svn mv not showing the new makefile.standalone.in?  Can you post 
> the new file makefile.standalone.in, or comment on the changes?
>
There were no changes, so I'm not sure what happened.  I'll try it again 
on a clean checkout and see if the problem repeats.


> Also, I noticed that you changed the form of some of the makefile 
> variables from ':= 'to '='.  Was this intentional?  I'm not the best 
> person to comment on the differences between the two and whether that 
> makes a difference here.  Stefan, can you comment if it looks OK.
That was not intentional.  Thanks for catching that.  The patch has 
changed quite a bit as I discussed it with Stefan and we determined a 
better way to do it.  Expect a new patch Here Soon.

A new question arose however:  Some time ago, we chose to put the HPEC 
benchmarks in a subdirectory of benchmarks/, much like the other 
specialized benchmarks, yet we gave it its own makefile so that it would 
stand alone a bit better.  This seems odd now.  I propose we move it 
alongside benchmarks/, perhaps as benchmarks_hpec/ to make it a little 
more obvious than 'hpec_kernel'.  We'll then mirror this arrangement 
upon installation.  Is this ok with you?

Regards,

-- 
Don McCoy
don (at) CodeSourcery
(888) 776-0262 / (650) 331-3385, x712