Bug 1113822 - bzr doesn't honor /usr/share/locale
Summary: bzr doesn't honor /usr/share/locale
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: bzr
Version: 19
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Toshio Ernie Kuratomi
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1124528
TreeView+ depends on / blocked
 
Reported: 2014-06-27 03:00 UTC by Ralf Corsepius
Modified: 2014-07-30 08:47 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
: 1124528 (view as bug list)
Environment:
Last Closed: 2014-07-30 08:47:06 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Fix the install location of locale files (583 bytes, patch)
2014-07-29 17:15 UTC, Toshio Kuratomi
no flags Details | Diff

Description Ralf Corsepius 2014-06-27 03:00:00 UTC
Description of problem:
bzr installs it's i18n files (*.mo) into /usr/share/share/locale instead of 
/usr/share/locale.

Version-Release number of selected component (if applicable):
bzr-2.5.1-13.fc19
bzr-2.6.0-2.fc20
bzr-2.6.0-4.fc21

How reproducible:
Always.

Steps to Reproduce:
1. Examine the contents of the rpms

Actual results:
rpm -qlp bzr-2.6.0-4.fc21.x86_64.rpm | grep /usr/share/share
/usr/share/share/locale/ar/LC_MESSAGES/bzr.mo
..

Expected results:
bzr to install *.mo into /usr/share/locale

Comment 1 Toshio Kuratomi 2014-07-29 17:15:07 UTC
Created attachment 922235 [details]
Fix the install location of locale files

Comment 2 Toshio Kuratomi 2014-07-29 17:18:17 UTC
Found the issue in the setup.py file.  At first, I was tempted to change the command to install the files in the spec file from

  python setup.py install --install-data %{_datadir}

to 

  python setup.py install --install-data %{_prefix}

but that causes the man pages to be installed into %{_prefix} as well.  So does appear to be a bug in upstream's setup.py.  The attached patch should fix that.

Comment 3 Ondrej Oprala 2014-07-30 08:47:06 UTC
The issue is now fixed in f21 and rawhide


Note You need to log in before you can comment on or make changes to this bug.