Re: %pretrans [package name]

2020-12-07 Thread Jerry James
On Mon, Dec 7, 2020 at 3:04 PM Miro Hrončok  wrote:
> I have figured it out. The package indeed does not exist on ppc64le, which is
> where the SRPM was created in Koji.
>
> Wrap the scriptlet in %if %{with docs}.

/me smacks forehead

Thank you, Miro!  There's no telling how long I might have puzzled over this.
-- 
Jerry James
http://www.jamezone.org/
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org


Re: %pretrans [package name]

2020-12-07 Thread Miro Hrončok

On 12/7/20 9:59 PM, Jerry James wrote:

On Mon, Dec 7, 2020 at 11:37 AM Jerry James  wrote:

Sorry for the silence.  $DAYJOB kicked into high gear last week.
Okay, I will give this a try today.  Thanks for the help, Miro and
Kevin!


Nope, that didn't work either:

https://koji.fedoraproject.org/koji/taskinfo?taskID=56996523

build.log says:

error: line 1572: %pretrans -n sagemath-doc-en -p 
: package sagemath-doc-en does not exist

I don't remember if that was the same error I got with "%pretrans
doc-en -p " or not, but it seems likely that it was.  Any other
ideas?


I have figured it out. The package indeed does not exist on ppc64le, which is 
where the SRPM was created in Koji.


Wrap the scriptlet in %if %{with docs}.

--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org


Re: %pretrans [package name]

2020-12-07 Thread Jerry James
On Mon, Dec 7, 2020 at 11:37 AM Jerry James  wrote:
> Sorry for the silence.  $DAYJOB kicked into high gear last week.
> Okay, I will give this a try today.  Thanks for the help, Miro and
> Kevin!

Nope, that didn't work either:

https://koji.fedoraproject.org/koji/taskinfo?taskID=56996523

build.log says:

error: line 1572: %pretrans -n sagemath-doc-en -p 
: package sagemath-doc-en does not exist

I don't remember if that was the same error I got with "%pretrans
doc-en -p " or not, but it seems likely that it was.  Any other
ideas?
-- 
Jerry James
http://www.jamezone.org/
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org


Re: %pretrans [package name]

2020-12-07 Thread Jerry James
On Thu, Dec 3, 2020 at 4:40 PM Miro Hrončok  wrote:
> On 12/3/20 6:08 PM, Kevin Kofler via devel wrote:
> > Miro Hrončok wrote:
> >> Try this (note the -n):
> >>
> >>   %pretrans -n doc-en -p 
> >
> > That would have to be:
> > %pretrans -n sagemath-doc-en -p 
> > because -n takes the full subpackage name, not just the suffix. (That is
> > normally the entire point of -n, to allow arbitrary names for subpackages.)
>
> Oh, right. I've mixed up. Disregard my previous email entirely.

Sorry for the silence.  $DAYJOB kicked into high gear last week.
Okay, I will give this a try today.  Thanks for the help, Miro and
Kevin!
-- 
Jerry James
http://www.jamezone.org/
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org


Re: %pretrans [package name]

2020-12-03 Thread Miro Hrončok

On 12/3/20 6:08 PM, Kevin Kofler via devel wrote:

Miro Hrončok wrote:

Try this (note the -n):

  %pretrans -n doc-en -p 


That would have to be:
%pretrans -n sagemath-doc-en -p 
because -n takes the full subpackage name, not just the suffix. (That is
normally the entire point of -n, to allow arbitrary names for subpackages.)


Oh, right. I've mixed up. Disregard my previous email entirely.

--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org


Re: %pretrans [package name]

2020-12-03 Thread Kevin Kofler via devel
Miro Hrončok wrote:
> Try this (note the -n):
> 
>  %pretrans -n doc-en -p 

That would have to be:
%pretrans -n sagemath-doc-en -p 
because -n takes the full subpackage name, not just the suffix. (That is 
normally the entire point of -n, to allow arbitrary names for subpackages.)

Kevin Kofler
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org


Re: %pretrans [package name]

2020-12-02 Thread Miro Hrončok

On 12/2/20 1:04 AM, Jerry James wrote:

Hi all,

I need a little help.  Sagemath packages up through version 9.1
bundled mathjax (several times!  with 1 png file for every single
glyph in every single font!), which made the documentation very large.
When I built version 9.2, I attempted to unbundle mathjax, replacing
the relevant directories with symlinks.  This requires a %pretrans
scriptlet, as described here:

https://docs.fedoraproject.org/en-US/packaging-guidelines/Directory_Replacement/

My first attempt added a script starting with "%pretrans doc-en -p
", since sagemath-doc-en is the package containing the
directories-turned-symlinks.  I built that version successfully in
mock, installed the old RPMs into a mock chroot, and then successfully
upgraded to the new version without RPM file conflicts.  Success!

Except that when I tried to build that version in koji, the SRPM
creation step failed.  I unfortunately did not write down the error,
but it pointed to the %pretrans line in the spec, and I somehow
deduced from the error message that something did not understand the
package name just after %pretrans.  So I changed it to "%pretrans -p
" in the most recent commit to the sagemath package git
repository, hoping that that would be okay, because it would still run
before the RPM transaction, right? Well, that didn't work:
https://bugzilla.redhat.com/show_bug.cgi?id=1903009

So now what?  How does one get "%pretrans [subpackage] -p " to
pass muster with koji?  Am I missing a trick?


Try this (note the -n):

%pretrans -n doc-en -p 

See an example in here:

https://src.fedoraproject.org/rpms/python-notebook/blob/master/f/python-notebook.spec#_175

--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org