By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 941

Summary: Build issue: kmymoney: FTBFS when using --enable-pdf-doc
Product: TDE Reporter: Darrell <darrella>
Component: other (any)Assignee: Timothy Pearson <kb9vqf>
Status: RESOLVED FIXED    
Severity: major CC: bugwatch, darrella, slavek.banko
Priority: P5    
Version: R14.0.0 [Trinity]   
Hardware: Other   
OS: Other   
Compiler Version: TDE Version String:
Application Version: Application Name:
Attachments: remove recode calls from kmymoney

Description Darrell 2012-03-27 16:36:12 CDT
Although the kmymoney configure process checks for the installation of the recode bin file, when using --enable-pdf-doc the configure process does not override and force --disable-pdf-doc when recode is not found. Instead the build fails.

Installing recode allows using --enable-pdf-doc, or using --disable-pdf-doc, but the configure process should override and fix on-the-fly.
Comment 1 Darrell 2012-03-27 17:04:25 CDT
Correction: whether recode is installed is irrelevant. kmymoney FTBFS with or without recode being installed.

The build error:

make[4]: Entering directory `/dev/shm/kmymoney/doc/en'
if test -n "" ; then \
                if test x != x.; then \
                        for file in credits.docbook details-accounts.docbook  details-categories.docbook details-files.docbook details-formats.docbook  details-hbciofx.docbook details-impexp.docbook details-institutions.docbook details-integrity.docbook details-investments.docbook details-ledgers.docbook details-loans.docbook details-payees.docbook details-reconciliation.docbook details-reports.docbook details-schedules.docbook details-search.docbook details-settings.docbook details-transactions.docbook details-widgets.docbook firsttime.docbook index.docbook introduction.docbook otherdoc.docbook views-accounts.docbook views-categories.docbook views-home.docbook views-investments.docbook views-ledgers.docbook views-outbox.docbook views-payees.docbook views-schedules.docbook views-institutions.docbook views-reports.docbook whatsnew.docbook details-currencies.docbook reference.docbook details-database.docbook makemostof.docbook details-budgets.docbook details-forecast.docbook; do \
                                cp ./$file .; \
                                chmod +w $file; \
                                if test -n ; then \
                                        recode utf-8.. $file; \
                                fi \
                        done; \
                        for file in accountdropdown.png accounts_view.png newacct.png accwiz1.png accwiz2.png accwiz3.png accwiz4.png accwiz5.png accwiz6.png accwiz7.png accwiz8.1.png accwiz8.2.png accwiz8.3.png accwiz8.4.png accwiz8.5.png accwiz9.png accwiz10.png calendarview.png categories.png categories_view.png categoryedit.png confirmenter.png default_prompt.png enterschedule.png finddlg.png home_view.png institutions_view.png investments_view.png kmt.png ledgers.png ledgers_view.png ledgerview-numbered.png mainwindow_numbered.png newfile.png newinst.png new.png newsched_numbered.png outbox_view.png payees_view.png payeeview.png qifopen.png reconcile.png reports_view.png schedcaltypes.png schedpopup.png schedule_view.png scheduleview.png tipofday.png transaction_find.png transactionform.png translist.png schedmonthday.png find-account.png find-amount.png find-category.png find-date.png find-details.png find-payee.png find-text.png payee_widget.png category_widget.png value_widget.png date_widget.png institution_view.png institution_options.png account_options.png account_edit.png payee_info.png payee_match.png payee_history.png split_transaction.png cleared_state.png transactionform-off.png ledger_more.png select_currency.png reconcile1.png reconcile2.png reconcile3.png reconcile4.png qif_report.png qifimportverify.png investment-transactionform.png investment-onlineupdate.png investment-currencywarning.png investment-exchangerateeditor.png gnucash-import_options.png gnucash-report.png gnucash-select_price_source.png currency_newpriceentry.png currency_priceeditor.png investments_summarytab.png qifimport-export.png qifimport-qifprofiledate.png qifimport-qifprofileeditor.png report_configuration-reporttab.png report_configuration-rowscolumns.png reports_view-all.png select_database.png budgets_view.png forecast_view.png newfile-2.png newfile-3.png newfile-4.png newfile-5.png newfile-6.png; do \
                                cp ./$file .; \
                                chmod +w $file; \
                        done; \
                fi \
        fi
make[4]: Leaving directory `/dev/shm/kmymoney/doc/en'
/opt/trinity/bin/meinproc --check --stylesheet `dirname /opt/trinity/share/apps/ksgmltools2/customization/kde-chunk.xsl`/kde-nochunk.xsl  ./index.docbook -o index.html;
recode utf8..latin1 index.html
recode: index.html failed: Invalid input in step `UTF-8..ISO-8859-1'
make[3]: *** [kmymoney-user.pdf] Error 1
Comment 2 Timothy Pearson 2012-05-13 17:41:24 CDT
Created attachment 621 [details]
remove recode calls from kmymoney

I am not convinced that recode should even be used in the build process.  Try the attached patch, and if it works (i.e. kmymoney builds and the resultant PDF is not corrupt) please push it to GIT.

Tim
Comment 3 Darrell 2012-06-12 23:57:03 CDT
The package builds with the attached patch and --enable-pdf-docs.

There is an improved patch --- looks like Francois is holding out on us: :-)

http://git.trinitydesktop.org/cgit/tde-packaging/tree/redhat/applications/kmymoney/kmymoney-3.5.13-recode_ftbfs.patch

I tested Francois' patch and that allows kmymoney to build with or without recode.

Although as a note to packagers, both html2ps and ps2pdf are build dependencies when using --enable-pdf-docs.

Two PDFs are generated. The PDFs are not copied to the package directory by make install. The work around is editing the build script to copy the PDFs to the desired location, but seems odd the PDFs are not automatically installed to the package directory.

Francois' patch pushed in GIT hash 672550ca.

Resolved!