Re: Problem packaging gnujump for practice

2015-06-22 Thread Theodotos Andreou

On 23/06/2015 06:09 πμ, Paul Wise wrote:

On Tue, Jun 23, 2015 at 1:44 AM, Theodotos Andreou wrote:


Filed. Let's see...

https://savannah.gnu.org/bugs/index.php?45375


A correction, the user config file was not generated during the
compile, it was generated when you ran gnujump for the first time.



Oh dear! I submitted a comment to correct it.


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/5588f4ba.3040...@ubuntucy.org



Re: Problem packaging gnujump for practice

2015-06-22 Thread Paul Wise
On Tue, Jun 23, 2015 at 1:44 AM, Theodotos Andreou wrote:

> Filed. Let's see...
>
> https://savannah.gnu.org/bugs/index.php?45375

A correction, the user config file was not generated during the
compile, it was generated when you ran gnujump for the first time.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/CAKTje6Hun8jx0cRpTFdf6y-UFG4ni7tJ=jy0i-wgjerx63f...@mail.gmail.com



Re: Problem packaging gnujump for practice

2015-06-22 Thread Theodotos Andreou

On 22/06/2015 06:13 μμ, Paul Wise wrote:

On Mon, Jun 22, 2015 at 11:07 PM, Theodotos Andreou wrote:


Do you think we should file a bug?


Filing a bug upstream sounds good to me, please mention the issue you
faced as a result of the design.



Filed. Let's see...

https://savannah.gnu.org/bugs/index.php?45375


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/5588498e.3060...@ubuntucy.org



Re: Problem packaging gnujump for practice

2015-06-22 Thread Paul Wise
On Mon, Jun 22, 2015 at 11:07 PM, Theodotos Andreou wrote:

> Do you think we should file a bug?

Filing a bug upstream sounds good to me, please mention the issue you
faced as a result of the design.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/caktje6fb5tv6m9cpfkghspurdmarcbthcmpgjfoe00pcj18...@mail.gmail.com



Re: Problem packaging gnujump for practice

2015-06-22 Thread Theodotos Andreou

On 22/06/2015 05:45 μμ, Paul Wise wrote:

On Mon, Jun 22, 2015 at 10:31 PM, Theodotos Andreou wrote:


It was hardcoded in the ~/.gnujump/ config. After I removed the config dir,
the error went away


That sounds like quite bad design, the binary shouldn't default copy
values from the binary into ~/.gnujump/ because they could be wrong if
you switch from the distro version to a self-compiled version and
vice-versa. Personally I would say it should not write a config file
at all unless you make changes from the default using the GUI.



Do you think we should file a bug?


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/558824ae.2040...@ubuntucy.org



Re: Problem packaging gnujump for practice

2015-06-22 Thread Paul Wise
On Mon, Jun 22, 2015 at 10:31 PM, Theodotos Andreou wrote:

> It was hardcoded in the ~/.gnujump/ config. After I removed the config dir,
> the error went away

That sounds like quite bad design, the binary shouldn't default copy
values from the binary into ~/.gnujump/ because they could be wrong if
you switch from the distro version to a self-compiled version and
vice-versa. Personally I would say it should not write a config file
at all unless you make changes from the default using the GUI.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/CAKTje6FaC55h8SGLXsYy=3x1eosm8p0xwvt2znbsg76zovv...@mail.gmail.com



Re: Problem packaging gnujump for practice

2015-06-22 Thread Theodotos Andreou

On 22/06/2015 01:26 μμ, Andrey Rahmatullin wrote:

On Mon, Jun 22, 2015 at 01:04:47PM +0300, Theodotos Andreou wrote:

Why does the binary references "/usr/local"

Mayeb because it's hardcoded in the source.
You'll need to find out +how is this path formed in the source and then
think how can that be fixed.



I figured it out eventually. It was hardcoded in the gnujump personal 
config file under ~/.gnujump/. After I had removed it, it went away.


Thanks for the help Andrey

--
theo


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/55881a68.3010...@theo-andreou.org



Re: Problem packaging gnujump for practice

2015-06-22 Thread Theodotos Andreou

On 22/06/2015 05:24 μμ, Jakub Wilk wrote:

* Theodotos Andreou , 2015-06-22, 13:04:

$ gnujump
Loading theme: /usr/local/share/gnujump/skins/default/config.theme
ERROR: Can open theme file
(/usr/local/share/gnujump/skins/default/config.theme).


Please upload your source package somewhere, so that we can have a look
at it.



It was hardcoded in the ~/.gnujump/ config. After I removed the config 
dir, the error went away


Thanks Jakub

--
theo


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/55881c2e.80...@ubuntucy.org



Re: Problem packaging gnujump for practice

2015-06-22 Thread Jakub Wilk

* Theodotos Andreou , 2015-06-22, 13:04:

$ gnujump
Loading theme: /usr/local/share/gnujump/skins/default/config.theme
ERROR: Can open theme file 
(/usr/local/share/gnujump/skins/default/config.theme).


Please upload your source package somewhere, so that we can have a look 
at it.


--
Jakub Wilk


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20150622142437.ga1...@jwilk.net



Re: Problem packaging gnujump for practice

2015-06-22 Thread Theodotos Andreou

On 22/06/2015 01:26 μμ, Andrey Rahmatullin wrote:

On Mon, Jun 22, 2015 at 01:04:47PM +0300, Theodotos Andreou wrote:

Why does the binary references "/usr/local"

Mayeb because it's hardcoded in the source.
You'll need to find out +how is this path formed in the source and then
think how can that be fixed.



You will not believe what the problem was. It was hardcoded in a config 
file under ~/gnujump. This file was initially created after I had built 
it using the traditional upstream tools and didn't change the prefix.


I figured this out after I installed the official gnujump from repos and 
it had the same issue.


So after removing ~/gnujump/ the problem went away.

Thank for the help.

--
theo


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/558819cc.9020...@ubuntucy.org



Re: Problem packaging gnujump for practice

2015-06-22 Thread Andrey Rahmatullin
On Mon, Jun 22, 2015 at 01:04:47PM +0300, Theodotos Andreou wrote:
> Why does the binary references "/usr/local"
Mayeb because it's hardcoded in the source. 
You'll need to find out +how is this path formed in the source and then
think how can that be fixed.

-- 
WBR, wRAR


signature.asc
Description: Digital signature


Problem packaging gnujump for practice

2015-06-22 Thread Theodotos Andreou

Dear mentors,

I am experimenting with debian packaging and I was following Lucas 
practical session 2 about packaging gnujump from upstream. The package 
is packed as a single package. (No gnujump-data).


The package has been built successfully but when I run the program I get:

$ gnujump
Loading theme: /usr/local/share/gnujump/skins/default/config.theme
ERROR: Can open theme file 
(/usr/local/share/gnujump/skins/default/config.theme).


The theme is in fact, under 
"/usr/share/gnujump/skins/default/config.theme" instead.


Why does the binary references "/usr/local"

In "debian/rules" I have:

```
DH_VERBOSE = 1

DPKG_EXPORT_BUILDFLAGS = 1
include /usr/share/dpkg/default.mk

export DEB_BUILD_MAINT_OPTIONS = hardening=+all

export DEB_CFLAGS_MAINT_APPEND  = -Wall -pedantic
export DEB_LDFLAGS_MAINT_APPEND = -Wl,--as-needed

%:
dh $@  --with autotools-dev

CFLAGS += -lm
```

Am I missing something obvious here?

You can find the complete build log here:

http://pastebin.com/WzjCRVzf

As you can see there is no reference to "/usr/local" anywhere.

Any Ideas?

--
theo


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/5587ddbf.6050...@ubuntucy.org