guile-strings.c; was: r13281 - gnucash/trunk - Make the build work if the build directory is not the source

[Derek Atkins]

Sorry 'bout that..  Didn't know that was a requirement.  Fixed in 13291.

-derek

Quoting Christian Stimming <stimming at tuhh.de>:

> Hi Derek,
>
> although the change in itself is very nice and a good improvement, 
> there's now
> one new issue introduced through this: The auto-generated list of
> translations strings from scheme files, intl-scm/guile-strings.c , includes
> the file name for each translation string. This is a workaround in order to
> give translators the information of which scheme file each string came from
> at least in the comment above a translation string, and it's proven quite
> useful. (To check: go into intl-scm directory, "rm guile-strings.c; make" and
> then look into guile-strings.c)
>
> It used to contain this file name relative to top_srcdir, but with 
> your change
> it will now contain a full absolute path name to those files. The old style
> meant that the po files which were generated from this guile-strings.c
> contained a relative path in the translator comments, independent of the
> actual build directory, which is just fine for po files that are committed to
> SVN and distributed in the tarball. With your change, the po files will now
> contain an absolute path in the translator comments, depending of the
> developer's current build directory, which is also committed to SVN and
> distributed in the tarball. I think that's ... unfortunate.
>
> (In the current update-po commit, I had manually removed the absolute path in
> guile-strings.c to avoid the confusing translators comments.)
>
> One possibility might be to pass the abs_top_srcdir variable to xgettext.scm
> (maybe as first argument), and then have it truncate the filenames by
> removing the leading x characters, where x is the string length of
> abs_top_srcdir; unfortunately my scheme-fu is too limited at the moment...
>
> Christian
>
> Am Freitag, 17. Februar 2006 21:28 schrieb Derek Atkins:
>> Author: warlord
>> Date: 2006-02-17 15:22:54 -0500 (Fri, 17 Feb 2006)
>> New Revision: 13281
>> Trac: http://svn.gnucash.org/trac/changeset/13281
>>
>> Modified:
>>    gnucash/trunk/ChangeLog
>>    gnucash/trunk/intl-scm/Makefile.am
>>    gnucash/trunk/lib/guile-www/Makefile.am
>>    gnucash/trunk/src/app-utils/Makefile.am
>>    gnucash/trunk/src/business/business-core/Makefile.am
>>    gnucash/trunk/src/business/business-gnome/Makefile.am
>>    gnucash/trunk/src/business/business-reports/Makefile.am
>>    gnucash/trunk/src/business/business-utils/Makefile.am
>>    gnucash/trunk/src/business/dialog-tax-table/Makefile.am
>>    gnucash/trunk/src/core-utils/Makefile.am
>>    gnucash/trunk/src/engine/Makefile.am
>>    gnucash/trunk/src/gnc-module/Makefile.am
>>    gnucash/trunk/src/gnome-utils/Makefile.am
>>    gnucash/trunk/src/gnome/Makefile.am
>>    gnucash/trunk/src/import-export/Makefile.am
>>    gnucash/trunk/src/import-export/hbci/Makefile.am
>>    gnucash/trunk/src/import-export/qif-import/Makefile.am
>>    gnucash/trunk/src/import-export/qif-io-core/Makefile.am
>>    gnucash/trunk/src/register/register-core/Makefile.am
>>    gnucash/trunk/src/report/locale-specific/us/Makefile.am
>>    gnucash/trunk/src/report/report-gnome/Makefile.am
>>    gnucash/trunk/src/report/report-system/Makefile.am
>>    gnucash/trunk/src/report/standard-reports/Makefile.am
>>    gnucash/trunk/src/report/stylesheets/Makefile.am
>>    gnucash/trunk/src/report/utility-reports/Makefile.am
>>    gnucash/trunk/src/scm/Makefile.am
>>    gnucash/trunk/src/scm/gnumeric/Makefile.am
>>    gnucash/trunk/src/scm/printing/Makefile.am
>>    gnucash/trunk/src/tax/us/Makefile.am
>> Log:
>>           Make the build work if the build directory is not the source
>>           directory.  Also symlink .scm files into the build directory
>>           so it's possible to run from it.
>> 	  Also fix "fin.scm installed twice", bug #168629.
>>
>>
>>
>> _______________________________________________
>> gnucash-patches mailing list
>> gnucash-patches at gnucash.org
>> https://lists.gnucash.org/mailman/listinfo/gnucash-patches
> _______________________________________________
> gnucash-devel mailing list
> gnucash-devel at gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-devel
>



-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available