We have moved to a new Sailfish OS Forum. Please start new discussions there.
5

Harbour rejects rpm filenames with .jolla., built with mer OBS [answered]

asked 2015-01-20 13:45:45 +0300

kimmoli gravatar image

updated 2015-01-20 16:01:24 +0300

eric gravatar image

When building sailfish applications on mer OBS, it generates RPM filenames that have text jolla in them between release and architecture

These were accepted still during week starting 13th Jan 2015, but not anymore.

e.g. harbour-chargemon-0.3.2-10.7.1.jolla.armv7hl.rpm now gets rejected (week ago similar was approved)

I would like to build my apps on mer OBS and publish them "as is" without renaming the filename.

I have studied the biary guidelines, and they clearly say that this is not allowed naming: image description

But still this s**ks.

I'm just interested what are the reasons behind for restrict naming of packages like this? and specially why it has been now changed to reject them automatically?

Also is there a way to configure mer OBS so that it does not add jolla in the RPM filename?

Or is it just me being stupid and not getting it?

edit retag flag offensive reopen delete

The question has been closed for the following reason "the question is answered, an answer was accepted" by kimmoli
close date 2015-01-22 15:54:00.391176

Comments

2

mer OBS was fixed to build w/o jolla by adding this to Project Configuration Release: <CI_CNT>.<B_CNT>

Now the filename is harbour-chargemon-0.3.2-8.1.armv7hl.rpm

(Workaround before @lbt gets it fixed for good)

kimmoli ( 2015-01-20 14:47:44 +0300 )edit

@eric I reopened this question as i'm still waiting for answer from Jolla, which was promised by @iekku after SFOS meeting 20th Jan 2015

kimmoli ( 2015-01-22 09:50:51 +0300 )edit

@kimmoli: Ok, sorry I didn't catch that one...

eric ( 2015-01-22 10:53:30 +0300 )edit

I think lbt's is all we can provide currently. We can close this as answered once lbt has done the fix.

iekku ( 2015-01-22 11:05:52 +0300 )edit

I don't think the fix has been applied by default as I just had to apply the kimmolis workaround to get rid of the "jolla" substring in my modRana Harbour packages.

MartinK ( 2015-05-16 01:54:32 +0300 )edit

2 Answers

Sort by » oldest newest most voted
6

answered 2015-01-22 14:39:30 +0300

Keto gravatar image

Regarding the naming restrictions in Harbour, it's basically about preventing name clashes with current and future system packages, and making the version comparisons a bit easier.

The 'harbour-' prefix makes sure that the store package names do not collide with system packages. For current system packages we could of course just check it, but as we don't know what the future might bring, it is easier to just require the prefix.

For the version we could accept anything RPM accepts, but the version comparison algorithm is quite complicated. And as we can't just use the comparison algorithm RPM library provides in all places we need to compare package versions, it was easier to restrict the version format so that the comparison is simpler to implement.

And why this happens now? Well, earlier the automatic check was just not as strict as we intended. The change should have been communicated more clearly, but it was assumed that there shouldn't be any effect on existing applications, so it was not included in the recent Harbour related announcements.

I hope this clarifies the situation.

edit flag offensive delete publish link more

Comments

Approved clarification, thanks!

and.. proving once more that assumption is mother of...

kimmoli ( 2015-01-22 15:58:48 +0300 )edit
4

answered 2015-01-20 14:49:20 +0300

lbt gravatar image

So this is because when I setup the SailfishOS projects in Mer OBS I'm using a copy of the internal configuration.

That kinda hardcodes a couple of values into the rpm name.

I'll fix the way I import the SailfishOS into the community OBS

In the meantime thanks for helping verify the fix kimmoli

edit flag offensive delete publish link more

Question tools

Follow
2 followers

Stats

Asked: 2015-01-20 13:45:45 +0300

Seen: 415 times

Last updated: Jan 22 '15