| mal | b100dian: obs doesn't support downloading anything during build, it can only download the package sources itself and of course dependencies but no downloads from within actual build | 00:43 |
|---|---|---|
| mal | b100dian: normal way of packaging projects is to add the upstream repo as submodule and then just the rpm folder with spec, at least when only packaging is needed without a lot of changes to upstream | 00:45 |
| b100dian | +1 thanks that sounds like a great idea for the 2nd problem | 00:46 |
| b100dian | for the 1st- I think there should be other _service(s) other than tar_git, that would make it possible to download from other type of sources - this one sounds like it would do https://github.com/openSUSE/obs-service-git_tarballs | 00:46 |
| mal | b100dian: an example of a repo with submodule https://github.com/sailfishos/libselinux | 00:47 |
| b100dian | There were also services like download_files or download_url mentioned in the OBS docs | 00:47 |
| b100dian | mal: yes, that was not something that crossed my mind but repo + rpm + submodule I see how this works now, thanks! | 00:48 |
| b100dian | oh, git_tarballs is 9y old, maybe that's not what I mean - I mean like tar_git, but without git:) Outside of the RPM spec | 00:49 |
| mal | b100dian: https://sailfishos.org/wiki/Usage_of_packaging_formats | 00:49 |
| mal | b100dian: hmm, trying to understand that other issue you have | 00:50 |
| mal | b100dian: that ldc service file is not really nice | 00:51 |
| b100dian | I would like to not have to upload a tar.gz in cases there's one on the web - | 00:51 |
| b100dian | that ldc should be pulled from https://github.com/ldc-developers/ldc/releases/tag/v1.26.0 | 00:52 |
| b100dian | well - to be fair, it should be compiled from sources | 00:52 |
| b100dian | but ldc needs ldc to be built | 00:52 |
| b100dian | so I didn't go the whole route (bootstrap ldc from gcc, then current ldc built by ldc) | 00:53 |
| mal | not sure if that kind of thing is supported, I need to ask the obs maintainer | 00:53 |
| b100dian | It should, see ldc_bootstrap here https://build.opensuse.org/package/view_file/devel:languages:D/ldc/ldc.spec | 00:55 |
| mal | b100dian: would bootstrap like this work: create temporary package of binary upload of ldc (with some other name), then create ldc package and it should see the temporary package for dependency and then remove the temp package after normal one has built | 00:56 |
| b100dian | I think you nailed it | 00:56 |
| b100dian | wait, not binary upload | 00:57 |
| b100dian | the temporary package -> C++ compiled # v0.17.6 is the last version buildable with a C++ compiler, so use it for bootstrapping | 00:57 |
| *** amccarthy_ is now known as amccarthy | 01:00 | |
| b100dian | I should probably try the whole bootstrapping some other time. What didn't you like about ldc, the %files section?:) | 01:00 |
| b100dian | anyway, there's still time tomorrow. thanks for looking into this, I'll have to go, it's late here and there too:) | 01:05 |
| mal | b100dian: I meant the _service file, which was completely commented you, you don't need one if you upload files like that | 01:12 |
| ss | What are consider flagship Sailfish phones? Judging by the website it seems like the Xperia 10, 10 II, and XA2 are preferred, but those seem like older devices. | 03:06 |
| pherjung | ss: officialy, there are no more recent devices than these | 03:44 |
| piggz | b100dian: hi, will you be working on fixing up patchmanager? | 09:06 |
| b100dian | piggz: you mean making it work on 4.x? I made a request for what I have already hacked as seen here https://forum.sailfishos.org/t/patchmanager-patches-in-koli-4-0-1/4925/109 | 09:22 |
| b100dian | or you mean the /etc/firejail/whitelist-common.local fix? | 09:23 |
| b100dian | mal: yes, ldc _service has different services I've tried commented out. It is not needed no | 09:25 |
| b100dian | *now | 09:25 |
| b100dian | ooh I think I get it - chum needs compatibility across 3.1->4.1 | 09:37 |
| b100dian | I think the git tag was in the wrong state when I made the request piggz, I corrected it immediately after that but it was probably cached with the old version that very moment | 09:39 |
| b100dian | now what we should be seeing is build failure for older releases.. | 09:41 |
| b100dian | piggz: done | 09:44 |
| testfun | thanks for the answers! | 11:36 |
| piggz | b100dian: im not sure what needs to be done for pm on 4, but good to see it building | 11:47 |
| b100dian[m] | I've also tested the build, but only on 4.0 and 4.1, wonder if I should disable 3.4 since I'm not able to test and there is a working one in openrepos. | 11:48 |
| ss | pherjung: I see, thank you. Is there one that seems to be a community favorite? | 12:20 |
| attah | ss: not really... out of those, all are good choices | 12:52 |
| attah | of course the 10 II is the must future-proof, but being on the initial release software, not all apps being available for aarch64 and it costing upwards of 2X the others... XA2 and 10 (I) are still valid choices | 12:55 |
| attah | Any ideas what might be causing this in OBS: ERROR: Source filename in .spec must end in .tar.gz, .tgz, .tar.bz2 or .tar.xz ? | 15:29 |
| attah | It builds fine locally, i don't have any source archive nor do i particularly want one... and adding the generic Source0: %{name}-%{version}.tar.bz2 does not do anything | 15:30 |
| renalcalculus[m] | attah: Yes, the Source0: definition must have one of these at the end for the OB builder to work. | 15:30 |
| attah | i'll retry with it, an dreport back... | 15:31 |
| attah | renalcalculus[m]: so now i have this, still same error: https://github.com/attah/sailfish-erlang/blob/master/rpm/erlang.spec | 15:34 |
| renalcalculus[m] | Weird. maybe remove the empty revision tag from the service file. | 15:36 |
| attah | it's a copy from a working package... but i can try | 15:36 |
| attah | no dice | 15:37 |
| renalcalculus[m] | i see it. i think it doesn't like something about the spec, maybe it needs a proper version: set? just guessing here though. | 15:39 |
| attah | renalcalculus[m]: how do you mean proper? package_ver seems to resolve fine | 15:42 |
| renalcalculus[m] | I think OBS tries to construct the tarball filename from the tags, but doesn't fully parse the .spec so gets confused about the %globals | 15:43 |
| attah | renalcalculus[m]: hah, that did it! thanks | 15:45 |
| renalcalculus[m] | heh there we go! :) | 15:46 |
| renalcalculus[m] | what do you need erlang for? | 15:46 |
| attah | need? want! | 15:46 |
| attah | no, nothing in particular really... but it is my primary language nowadays, could come in handy to have it easily available for experiments | 15:47 |
| renalcalculus[m] | Ah I see. Nice. | 15:49 |
| renalcalculus[m] | Now you need a '%setup -n -q %{name}-%{version}/upstream' I think | 15:50 |
| attah | renalcalculus[m]: nice :) so does that make the silly cd command redundant or not? | 15:52 |
| renalcalculus[m] | it should, unless you have a upstream/upstream dir you want to patch in. | 15:53 |
| attah | i have it in %build and %install too... guess i can test that locally though | 15:54 |
| attah | hmm, sfdk prepare did not like that | 15:58 |
| renalcalculus[m] | I never used the SDK, so I'm not sure OBS and sfdk are compatible | 15:59 |
| attah | well, taht would suck... but surely there must be a common way to do it | 16:00 |
| renalcalculus[m] | hehe, sorry, it's %setup -q -n not -n -q | 16:03 |
| attah | right... that other stuff is a parameter to -n | 16:05 |
| attah | ah.. and now the cd is redundant | 16:07 |
| renalcalculus[m] | maybe you can do something like [ -e upstream] && pushd upstream if you need that for sfdk | 16:07 |
| attah | no, now it doesn't like that either | 16:08 |
| attah | with the proper setup, no cd is needed at any step it seems | 16:10 |
| attah | :) | 16:10 |
| attah | i'll just let the build finish locally before trying again | 16:11 |
| attah | and again, without systemd | 16:22 |
| attah | hmm, maybe i should have put that in BuildRequires, and not just dropped it... but good enough for testing | 16:24 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!