Re: [Stormpkg-devel] debian/rules + misc

[Christian K. <sh...@de...>]

On 00-07-18 Marcelo E. Magallon wrote:
> >> Christian Kurz <sh...@de...> writes:

>  > >  I didn't change this because this is not the reason I was granted write
>  > >  access to the CVS tree.  Would anyone mind if I change such things
>  > >  directly (with a proper log) or do you guys prefer to have a bug
>  > >  submitted?
>  > 
>  > No, I think as you got the same cvs-access and one is able to trace the
>  > changes of somebody, you should fold in the changes in the cvs-tree.

>  Oh, I know, sorry, I failed to express myself correctly.  Plaintext:
>  Does anyone here mind if I poke arround with stuff besides the
>  translation?

I think nobody would be offended, if you also help fixing and improving
stormpkg instead of only doing some translation stuff. This is our
common interest and as you have the same cvs permissions as every other
stormpkg developer, you should also poke around with the other stuff and
help creating the best gui package adminstration tool. ;)

>  > >  i.e., stuff is getting installed on $(prefix)/blah/$(datadir), and
>  > >  prefix is set to /usr.  One option is to install the files with prefix
>  > >  set to $PWD/debian/tmp/usr and the other one is applying this patch.
>  > 
>  > If the prefix is currently set to point to /usr, I think we should
>  > change the default so, that it points to /usr/local instead.

>  The default points to /usr/local (better said, the default is not
>  changed)

Ah, then my thaught that this is just a typo was right.

>  > Anyway I think the current behaviour is the same as for other
>  > software which comes out of cvs or a tar.gz. So I don't see the need
>  > for changing it, but if the others think it's good, then let's fold
>  > it in.

>  Oh, sorry again.

>  I mean debian/rules:

>          ./autogen.sh --prefix=/usr

>          $(MAKE) install DESTDIR=`pwd`/debian/tmp prefix=/usr
>  
>  on the install target, prefix is set to /usr and the files are moved to
>  the temporaty location using DESTDIR.  The makefile on po/ doesn't use
>  DESTDIR (any package using an unmodified gettext Makefile has this
>  "problem").  Either set prefix to `pwd`/debian/tmp/usr on the install
>  target (which is, in general, not a good idea -- some, badly written
>  IMHO, packages perform black magic on their install targets) or patch
>  the Makefile in po/ to use DESTDIR.

Okay, then you should commit the patch into the cvs, as the current
behaviour of the Makefile is not acceptable and can cause some trouble
for the users, so please fix this. 

>  > Well, I noticed some other stuff and so I cleaned the cvs-tree a bit
>  > more up and now it's hopefully clean enough. I only failed to remove the
>  > directory .xvpics under pixmaps. Could somebody with more knowledge
>  > about cvs do this please, I tried but somehow failed.
>  
>  You can't remve directories from CVS (unless, of course, you actually
>  remove the directory on the repository).  Use -P on update.

Thank your for the suggestion, but I already tried that. I intalled the
cvsbook and found their this solution but it didn't work as expected.
And I didn't want to remove the directory from the repository without
having contacted first the other developers.

>  > >  And lastly, for some reason my Makefile.in.in got changed/updated
>  > >  (sorry, I'm not at home to check the version the gettext -- it's
>  > >  whatever is on woody atm)
>  > 
>  > Which Makefile.in.in do you exactly mean, the one in po? Could you
>  > please look what the changes are and tell us?
>  
>  Yups po/Makefile.in.in

>  diff attached.  It's pretty harmless, it justs cleans generated files.

[...]

Good patch, please commit it also into the cvs. Thanks for nice work!

Ciao
     Christian
-- 
Ein "Nein" ausgesprochen mit der tiefsten Überzeugung ist besser
und größer als ein "Ja" um zu gefallen oder noch schlimmer, um
Schwierigkeiten zu umgehen.
  -- Mahatma Gandhi