Changes to debian version

classic Classic list List threaded Threaded
19 messages Options
Reply | Threaded
Open this post in threaded view
|

Changes to debian version

Damyan Ivanov
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi,

I am almost finished building debian package for FlameRobin.

I had to change the following:

1)  make configure not remove confdefs dir, but only confdefs.h

- --- ./configure 2005-09-05 12:16:31.000000000 +0300
+++ /tmp/dpep-work.Jj0eTi/flamerobin-0.4.0/configure    2005-09-16
09:54:51.000000000 +0300
@@ -1124,7 +1124,7 @@
     echo "$as_me: exit $exit_status"
   } >&5
   rm -f core *.core &&
- -  rm -rf conftest* confdefs* conf$$* $ac_clean_files &&
+  rm -rf conftest* confdefs.h conf$$* $ac_clean_files &&
     exit $exit_status
      ' 0
 for ac_signal in 1 2 13 15; do

2) Do not copy confdefs directory when building
- --- ./Makefile.in       2005-09-16 09:50:52.000000000 +0300
+++ /tmp/dpep-work.Jj0eTi/flamerobin-0.4.0/Makefile.in  2005-09-16
09:50:52.000000000 +0300
@@ -239,16 +239,17 @@
 @COND_PLATFORM_MACOSX_1@flamerobin_bundle:
$(____flamerobin_BUNDLE_TGT_REF_DEP)
 config-options: flamerobin$(EXEEXT)
- -       @mkdir -p $(COPYFILESDESTDIR)/confdefs
- -       @for f in fr_settings.confdef db_settings.confdef; do \
- -       if test ! -f $(COPYFILESDESTDIR)/confdefs/$$f -a ! -d
$(COPYFILESDESTDIR)/confdefs/$$f ; \
- -       then x=yep ; \
- -       else x=`find $(srcdir)/confdefs/$$f -newer
$(COPYFILESDESTDIR)/confdefs/$$f -print` ; \
- -       fi; \
- -       case "$$x" in ?*) \
- -       cp -pRf $(srcdir)/confdefs/$$f $(COPYFILESDESTDIR)/confdefs ;; \
- -       esac; \
- -       done
+
+#      @mkdir -p $(COPYFILESDESTDIR)/confdefs
+#      @for f in fr_settings.confdef db_settings.confdef; do \
+#      if test ! -f $(COPYFILESDESTDIR)/confdefs/$$f -a ! -d
$(COPYFILESDESTDIR)/confdefs/$$f ; \
+#      then x=yep ; \
+#      else x=`find $(srcdir)/confdefs/$$f -newer
$(COPYFILESDESTDIR)/confdefs/$$f -print` ; \
+#      fi; \
+#      case "$$x" in ?*) \
+#      cp -pRf $(srcdir)/confdefs/$$f $(COPYFILESDESTDIR)/confdefs ;; \
+#      esac; \
+#      done

The above two changes were to fix FR build in the top-level directory.
This is my choice to build it this way for Debian, since building in
another directory doesn't easily fin in debian build infrastructure (or,
I am not skilled enough to make it fit). Also, the distclean target does
great job in removing generated content.

3) fix hardcoded install prefix
- --- ./src/config/Config.cpp     2005-09-16 10:28:36.000000000 +0300
+++ /tmp/dpep-work.WQl3rs/flamerobin-0.4.0/src/config/Config.cpp
2005-09-16 10:29:00.000000000 +0300
@@ -62,7 +62,7 @@
     : homePathM(""), userHomePathM(""), configM(0)
 {
 #if defined(__UNIX__) && !defined(__WXMAC__) && !defined(__DARWIN__)
- -    standardPathsM.SetInstallPrefix(wxT("/usr/local"));
+    standardPathsM.SetInstallPrefix(wxT("/usr"));
 #endif
 }

This is because FR runtime does not honour configure's --prefix
parameter. Instead of findiing my way through autotools jungle, I've
decided to change the debian package version to use appropriate prefix.
I'll look into this later.

4) res/fricon.xpm:
Moved /* XPM */ header before the license. Without this change xpm lib
refuses to recognise the file as a valid XPM. Note that this file is
used to show FR in the desktop menu (Gnome/KDE/etc) and that is where
the problem shows. FR itself has no problem with such files, because it
uses them as raw C code, I guess.

This change can be troublesome, since it changes the positiion of the
license statement. Any objections? Legal stuff is so fragile, so I'd
better ask.

I am not proposing that the above changes are incorporated in FR
sources. No. I just wanted you to know what and why had to be changed in
case I am doing something wrong.


The packaging needs some cleanup that I'll do later today. After this
I'll give an URL for download.


dam
- --
Damyan Ivanov          0x9725F63B          Creditreform Bulgaria
[hidden email]              http://www.creditreform.bg/
phone: +359(2)928-2611, 929-3993            fax: +359(2)920-0994
mob. +359(88)856-6067  ICQ 3028500  [hidden email]/Gaim
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD4DBQFDKpA3Hqjlqpcl9jsRAnoDAJjPzQNBHmhXRvd/GfwNvctfrciVAJwP+lJp
2taq2a06q/uoMbsiuu9q/w==
=sDy4
-----END PGP SIGNATURE-----


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Milan Babuskov-5
Damyan Ivanov wrote:
> 1)  make configure not remove confdefs dir, but only confdefs.h
>
> - --- ./configure 2005-09-05 12:16:31.000000000 +0300
>
> 2) Do not copy confdefs directory when building

Please note that the configure script is built using bakefile
(flamerobin.bkl is the source), so any changes to it get lost. We need
to solve things like this in the .bkl file.

> 3) fix hardcoded install prefix
> - --- ./src/config/Config.cpp     2005-09-16 10:28:36.000000000 +0300
> +++ /tmp/dpep-work.WQl3rs/flamerobin-0.4.0/src/config/Config.cpp
>
> This is because FR runtime does not honour configure's --prefix
> parameter. Instead of findiing my way through autotools jungle, I've
> decided to change the debian package version to use appropriate prefix.
> I'll look into this later.

Michael is also working on this, so you better exchange the knowledge.

> 4) res/fricon.xpm:
> Moved /* XPM */ header before the license. Without this change xpm lib
> refuses to recognise the file as a valid XPM. Note that this file is
> used to show FR in the desktop menu (Gnome/KDE/etc) and that is where
> the problem shows. FR itself has no problem with such files, because it
> uses them as raw C code, I guess.
>
> This change can be troublesome, since it changes the positiion of the
> license statement. Any objections? Legal stuff is so fragile, so I'd
> better ask.

No problem with this. Is it only needed for fricon.xpm, or for all .xpm
files?

> I am not proposing that the above changes are incorporated in FR
> sources. No. I just wanted you to know what and why had to be changed in
> case I am doing something wrong.

Well, I'd like everything to be in a single place if possible, so next
time the package is built we don't need to patch the files.

> The packaging needs some cleanup that I'll do later today. After this
> I'll give an URL for download.

Great.

--
Milan Babuskov
http://fbexport.sourceforge.net
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Damyan Ivanov
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Milan Babuskov wrote:

> Damyan Ivanov wrote:
>
>> 1)  make configure not remove confdefs dir, but only confdefs.h
>>
>> - --- ./configure 2005-09-05 12:16:31.000000000 +0300
>>
>> 2) Do not copy confdefs directory when building
>
>
> Please note that the configure script is built using bakefile
> (flamerobin.bkl is the source), so any changes to it get lost. We need
> to solve things like this in the .bkl file.

I see no problem with patching configure directly because it is created
during *release* process and does not change during *buld*.

>> 3) fix hardcoded install prefix
>> - --- ./src/config/Config.cpp     2005-09-16 10:28:36.000000000 +0300
>> +++ /tmp/dpep-work.WQl3rs/flamerobin-0.4.0/src/config/Config.cpp
>>
>> This is because FR runtime does not honour configure's --prefix
>> parameter. Instead of findiing my way through autotools jungle, I've
>> decided to change the debian package version to use appropriate prefix.
>> I'll look into this later.
>
> Michael is also working on this, so you better exchange the knowledge.

Is there SVN/CVS repository where I can look what's already done?

>> 4) res/fricon.xpm:
>> Moved /* XPM */ header before the license. Without this change xpm lib
>> refuses to recognise the file as a valid XPM. Note that this file is
>> used to show FR in the desktop menu (Gnome/KDE/etc) and that is where
>> the problem shows. FR itself has no problem with such files, because it
>> uses them as raw C code, I guess.
>>
>> This change can be troublesome, since it changes the positiion of the
>> license statement. Any objections? Legal stuff is so fragile, so I'd
>> better ask.
>
>
> No problem with this. Is it only needed for fricon.xpm, or for all .xpm
> files?

Other .xpm files are not used outside FR, but making the change won't
hurt anyway.

>> I am not proposing that the above changes are incorporated in FR
>> sources. No. I just wanted you to know what and why had to be changed in
>> case I am doing something wrong.
>
> Well, I'd like everything to be in a single place if possible, so next
> time the package is built we don't need to patch the files.

Don't worry. You'll get all the patches plus the infrastructure that
applies/unapplies them. It would be extremely helpful to have a Debian
system at hand, though.


dam
- --
Damyan Ivanov          0x9725F63B          Creditreform Bulgaria
[hidden email]              http://www.creditreform.bg/
phone: +359(2)928-2611, 929-3993            fax: +359(2)920-0994
mob. +359(88)856-6067  ICQ 3028500  [hidden email]/Gaim
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFDKqoCHqjlqpcl9jsRArIPAJ0VMhvBKD22o9TCqlnfM3lBRa4UuwCeI+/a
kY7/hMahChfM7FDcakz0msc=
=UYfl
-----END PGP SIGNATURE-----


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Milan Babuskov-2
Damyan Ivanov wrote:
>>Please note that the configure script is built using bakefile
>>(flamerobin.bkl is the source), so any changes to it get lost. We need
>>to solve things like this in the .bkl file.
>
> I see no problem with patching configure directly because it is created
> during *release* process and does not change during *buld*.

Well, as soon as we add a new file, configure script is regenerated...
or is it not? I see that Makefile.in changes, but I'm not sure when does
the configure change. Michael, could you give me a hand here?

Anyway, I believe the configure script does get regenrated at some
point, and your changes will be lost. No harm done "durnig build", but
why not make things right once, and have them working always...

>>>This is because FR runtime does not honour configure's --prefix
>>>parameter. Instead of findiing my way through autotools jungle, I've
>>>decided to change the debian package version to use appropriate prefix.
>>>I'll look into this later.
>>
>>Michael is also working on this, so you better exchange the knowledge.
>
> Is there SVN/CVS repository where I can look what's already done?

There is, but since you're not a member of project (yet) you only have
anonymous access, which lags few hours (sometimes days) behind. I think
is much better if you guys discuss it here.

>>No problem with this. Is it only needed for fricon.xpm, or for all .xpm
>>files?
>
> Other .xpm files are not used outside FR, but making the change won't
> hurt anyway.

Ok.

> Don't worry. You'll get all the patches plus the infrastructure that
> applies/unapplies them. It would be extremely helpful to have a Debian
> system at hand, though.

I'm writing this on Mepis, which is a Debian derivative, and AFAIK uses
the Sid packages. At least, I see "Debian Thunderbird" as the title of
my e-mail client. I hope that will do.

I also plan to install Sarge on my home computer.

--
Milan Babuskov
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Damyan Ivanov
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Milan Babuskov wrote:
> Damyan Ivanov wrote:
>> I see no problem with patching configure directly because it is created
>> during *release* process and does not change during *buld*.
>
> Well, as soon as we add a new file, configure script is regenerated...
> or is it not? I see that Makefile.in changes, but I'm not sure when does
> the configure change. Michael, could you give me a hand here?

You can't add new file after release :-)

The Debian packaging works on expanded upstream-source.tar.gz, which
already contains configure. And this same configure is not regenerated
during build. Only patched if necessary to cope with Debian requirements.

> Anyway, I believe the configure script does get regenrated at some
> point, and your changes will be lost. No harm done "durnig build", but
> why not make things right once, and have them working always...

I was not sure whether the changes are acceptable for other platforms.
So I added a patch (or two) to the list of patches that get applied
before ./configure ... make ... make install are run. It is simple,
works and doesn't require upstream to change anything.

Well, if the patch is considered useful for other platforms too, that's
another story.

>> Don't worry. You'll get all the patches plus the infrastructure that
>> applies/unapplies them. It would be extremely helpful to have a Debian
>> system at hand, though.
>
> I'm writing this on Mepis, which is a Debian derivative, and AFAIK uses
> the Sid packages. At least, I see "Debian Thunderbird" as the title of
> my e-mail client. I hope that will do.
>
> I also plan to install Sarge on my home computer.

Both may do the job.

But it would be much easier if flamerobin enters "mainstream" Debian
archive. I can try to do this if the project does not mind.


Debian packages (source and i386 binary) are on
ftp://ftp.logos-bg.net/debian-addons-bg/dists/sid/main/flamerobin

Binary packages are suitable only for recent Sid systems and won't
install on plain sarge.

Also there are the build logs.

All necessary files for building a Debian package are in .diff.gz file,
that should be applied to an expanded upstream tar.gz. patches are in
debian/patches/


dam
- --
Damyan Ivanov          0x9725F63B          Creditreform Bulgaria
[hidden email]              http://www.creditreform.bg/
phone: +359(2)928-2611, 929-3993            fax: +359(2)920-0994
mob. +359(88)856-6067  ICQ 3028500  [hidden email]/Gaim
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFDKtWsHqjlqpcl9jsRAkDqAKCqZapqjECggD+nuqOI0U7VP7HuogCgrjyb
9eLZVSfmRVkgKXd1qz7i3Lo=
=Omds
-----END PGP SIGNATURE-----


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Milan Babuskov-2
Damyan Ivanov wrote:
>>Well, as soon as we add a new file, configure script is regenerated...
>>or is it not? I see that Makefile.in changes, but I'm not sure when does
>>the configure change. Michael, could you give me a hand here?
>
> You can't add new file after release :-)

Of course :) I meant "for the next release".

> The Debian packaging works on expanded upstream-source.tar.gz, which
> already contains configure. And this same configure is not regenerated
> during build. Only patched if necessary to cope with Debian requirements.

I see. Ok.

> I was not sure whether the changes are acceptable for other platforms.
> So I added a patch (or two) to the list of patches that get applied
> before ./configure ... make ... make install are run. It is simple,
> works and doesn't require upstream to change anything.
>
> Well, if the patch is considered useful for other platforms too, that's
> another story.

We'll see after we check them out. Thanx.

> But it would be much easier if flamerobin enters "mainstream" Debian
> archive. I can try to do this if the project does not mind.

Please do.

--
Milan Babuskov
http://fbexport.sourceforge.net
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Milan Babuskov-2
In reply to this post by Damyan Ivanov
Damyan Ivanov wrote:
> Debian packages (source and i386 binary) are on
> ftp://ftp.logos-bg.net/debian-addons-bg/dists/sid/main/flamerobin

Actually, it is:
ftp://ftp.logos-bg.net/debian-addons-bg/dists/sid/flamerobin/

--
Milan Babuskov
http://fbexport.sourceforge.net
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Michael Hieke
In reply to this post by Damyan Ivanov
Damyan, all,

Damyan Ivanov wrote:

> I had to change the following:
>
> 1)  make configure not remove confdefs dir, but only confdefs.h
>
> - --- ./configure 2005-09-05 12:16:31.000000000 +0300
> +++ /tmp/dpep-work.Jj0eTi/flamerobin-0.4.0/configure    2005-09-16
> 09:54:51.000000000 +0300
> @@ -1124,7 +1124,7 @@
>      echo "$as_me: exit $exit_status"
>    } >&5
>    rm -f core *.core &&
> - -  rm -rf conftest* confdefs* conf$$* $ac_clean_files &&
> +  rm -rf conftest* confdefs.h conf$$* $ac_clean_files &&
>      exit $exit_status
>       ' 0
>  for ac_signal in 1 2 13 15; do

This is a very important point.  Since I don't think it's a good thing
that we would also need to patch the configure script - maybe we should
change the name of the confdefs directory.  That's easy since we haven't
moved the CVS yet.

> 2) Do not copy confdefs directory when building

The copy actions will be removed from flamerobin.bkl, so it should no
longer be necessary then.

Thanks

--
Michael Hieke




-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Michael Hieke
In reply to this post by Milan Babuskov-2
Milan,

Milan Babuskov wrote:

> Well, as soon as we add a new file, configure script is
> regenerated... or is it not? I see that Makefile.in changes, but I'm
> not sure when does the configure change. Michael, could you give me a
> hand here?

when build files are rebaked, Makefile.in is recreated if needed.  The
configure script OTOH needs only be recreated when either
autoconf_inc.m4 (created from flamerobin.bkl) or configure.in were
changed.  That happens fairly seldom.  You need to run aclocal and
autoconf to recreate configure script.

--
Michael Hieke



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Nando Dessena
In reply to this post by Michael Hieke
Michael,

M> This is a very important point.  Since I don't think it's a good thing
M> that we would also need to patch the configure script - maybe we should
M> change the name of the confdefs directory.  That's easy since we haven't
M> moved the CVS yet.

do you mean that in complete ignorance and good faith I have picked a
name that clashes with some configure thing? Cute.

Ciao
--
Nando Dessena
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Damyan Ivanov
In reply to this post by Milan Babuskov-2
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Milan Babuskov wrote:
> Damyan Ivanov wrote:
>
>>> Well, as soon as we add a new file, configure script is regenerated...
>>> or is it not? I see that Makefile.in changes, but I'm not sure when does
>>> the configure change. Michael, could you give me a hand here?
>>
>> You can't add new file after release :-)
>
> Of course :) I meant "for the next release".

I see no problem with this.
Releasing new upstream version of the Debian package requires manual
intervention from Debian maintainer anyway. If the configure script has
changes and if the patch no longer applies cleanly, it is responsibility
of the maintainer to fix the patch.


dam
- --
Damyan Ivanov          0x9725F63B          Creditreform Bulgaria
[hidden email]              http://www.creditreform.bg/
phone: +359(2)928-2611, 929-3993            fax: +359(2)920-0994
mob. +359(88)856-6067  ICQ 3028500  [hidden email]/Gaim
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFDK75VHqjlqpcl9jsRAoG9AJ0RdnTskwoK59pOjBX3KU/n5m9B/gCggLDH
9jJFY4+ZoQMisf7cichJN/o=
=5L96
-----END PGP SIGNATURE-----


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Milan Babuskov-2
Damyan Ivanov wrote:

>>>>Well, as soon as we add a new file, configure script is regenerated...
>>>>or is it not? I see that Makefile.in changes, but I'm not sure when does
>>>>the configure change. Michael, could you give me a hand here?
>>>
>>>You can't add new file after release :-)
>>
>>Of course :) I meant "for the next release".
>
> I see no problem with this.
> Releasing new upstream version of the Debian package requires manual
> intervention from Debian maintainer anyway. If the configure script has
> changes and if the patch no longer applies cleanly, it is responsibility
> of the maintainer to fix the patch.

Ok. To be honest, I didn't know how the whole "maintainter" stuff works.
Thanx for clarification.

--
Milan Babuskov
http://fbexport.sourceforge.net
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Michael Hieke
In reply to this post by Nando Dessena
Nando,

Nando Dessena wrote:

>> This is a very important point.  Since I don't think it's a good
>> thing that we would also need to patch the configure script - maybe
>> we should change the name of the confdefs directory.  That's easy
>> since we haven't moved the CVS yet.
>
> do you mean that in complete ignorance and good faith I have picked a
> name that clashes with some configure thing? Cute.

it kind of looks like it.  The configure script uses compilation of test
programs to find whether some headers, types or compiler features are
available on a system, and removes all these temporary files when it's
done with them.  At least confdefs.h is one of those files, and there is
a wildcard delete with confdefs*, which would completely wipe out our
directory...
Maybe we can find a slightly different name that fits as well, it should
be easier then making configure behave...

Sorry, but I never noticed, I only ever build in subdirectories of
$(FR_HOME).

Thanks

--
Michael Hieke



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Nando Dessena
Michael,

M> Maybe we can find a slightly different name that fits as well, it should
M> be easier then making configure behave...

conf-templates comes to mind. Although in this case I'd like to change
the extension ".confdef" as well and I don't have a good substitute
right now.

Ciao
--
Nando Dessena
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Milan Babuskov-2
Nando Dessena wrote:
> Michael,
>
> M> Maybe we can find a slightly different name that fits as well, it should
> M> be easier then making configure behave...
>
> conf-templates comes to mind. Although in this case I'd like to change
> the extension ".confdef" as well and I don't have a good substitute
> right now.

Let's change the folder name to "configuration-definitions" or
"config-defs" and leave extension .confdef.

--
Milan Babuskov
http://fbexport.sourceforge.net
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Michael Hieke
Milan, Nando,

Milan Babuskov wrote:

> Let's change the folder name to "configuration-definitions" or
> "config-defs" and leave extension .confdef.

+1 from me for "config-defs".

Thanks

--
Michael Hieke


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Nando Dessena
Michael, Milan,

>> Let's change the folder name to "configuration-definitions" or
>> "config-defs" and leave extension .confdef.

M> +1 from me for "config-defs".

I'd prefer "conf-defs". Shortening the same word (configuration) in
two different ways (conf and config) in the same context doesn't look
nice.

Ciao
--
Nando Dessena
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Michael Hieke
Nando,

Nando Dessena wrote:
>> +1 from me for "config-defs".
>
> I'd prefer "conf-defs". Shortening the same word (configuration) in
> two different ways (conf and config) in the same context doesn't look
> nice.

+1 for that one too ;-)

Thanks

--
Michael Hieke


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel
Reply | Threaded
Open this post in threaded view
|

Re: Changes to debian version

Milan Babuskov-5
In reply to this post by Nando Dessena
Nando Dessena wrote:

> Michael, Milan,
>
>
>>>Let's change the folder name to "configuration-definitions" or
>>>"config-defs" and leave extension .confdef.
>
>
> M> +1 from me for "config-defs".
>
> I'd prefer "conf-defs". Shortening the same word (configuration) in
> two different ways (conf and config) in the same context doesn't look
> nice.

Ok, agreed.

--
Milan Babuskov
http://fbexport.sourceforge.net
http://www.flamerobin.org



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server.
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Flamerobin-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/flamerobin-devel