[openib-general] RE: [PATCH] osm: Adding revision empty file

Hal Rosenstock halr at voltaire.com
Wed Mar 1 08:23:51 PST 2006


On Wed, 2006-03-01 at 11:16, Sasha Khapyorsky wrote:
> On 10:44 Wed 01 Mar     , Hal Rosenstock wrote:
> > On Wed, 2006-03-01 at 10:43, Sasha Khapyorsky wrote:
> > > On 16:50 Wed 01 Mar     , Ofer Gigi wrote:
> > > >
> > > > To reproduce:
> > > > 
> > > > 	cd src/userspace/management/osm
> > > > 	./autogen.sh
> > > > 	./configure --disable-libcheck
> > > > 	make dist
> > > 
> > > Hal, Ofer, do we need distributible osm_svn_revision.h? If not we may just 
> > > remove it from dist and dep lists. If yes then probably we may move
> > > osm_svn_revision.h generation to include/Makefile.am.
> > > 
> > > To add autogenerated file under source control is not good idea.
> > 
> > That was what caused the problem in the first place.
> 
> Yes.
> 
> After more thinking: there is no big reason to make osm_svn_revision.h
> distributible - this will be recreated in build time anyway. Right?

Right. The dist make should cause it to be regenerated and I think it
would indicate exported.

-- Hal

> Ofer, Vladimir, could you try this:
> 
> 
> diff --git a/osm/include/Makefile.am b/osm/include/Makefile.am
> index 68e60bf..e1f7070 100644
> --- a/osm/include/Makefile.am
> +++ b/osm/include/Makefile.am
> @@ -4,7 +4,6 @@ nobase_pkginclude_HEADERS = iba/ib_types
>  
>  EXTRA_DIST = \
>  	$(srcdir)/opensm/osm_version.h \
> -	$(srcdir)/opensm/osm_svn_revision.h \
>  	$(srcdir)/opensm/osm_sa_portinfo_record_ctrl.h \
>  	$(srcdir)/opensm/osm_sa_guidinfo_record_ctrl.h \
>  	$(srcdir)/opensm/osm_sa_path_record.h \
> 
> 
> Just to be sure, I checked it with:
> 
>  ./autogen.sh && ./configure --disable-libcheck && make dist && make
> 
> but my dev tree may be dirty.
> 
> Sasha.




More information about the general mailing list