https://bugzilla.redhat.com/show_bug.cgi?id=1813563
Mattia Verga changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #17 from Gwyn Ciesla ---
(fedscm-admin): The Pagure repository was created at
https://src.fedoraproject.org/rpms/libpasraw
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #16 from Antonio T. (sagitter) ---
(In reply to Mattia Verga from comment #14)
> >
> > - %{forgesource} ??
> https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/
> #_using_forges_hosted_revision_control
>
Ah!
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
Antonio T. (sagitter) changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #14 from Mattia Verga ---
Spec URL: https://mattia.fedorapeople.org/libpasraw.spec
SRPM URL: https://mattia.fedorapeople.org/libpasraw-1.3.0-4.fc33.src.rpm
Description: Provides shared library to interface Pascal program with
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #13 from Fabio Valentini ---
> This library is loaded in Pascal source code with
> `librawname='libpasraw.so.1';` and `libraw := LoadLibrary(librawname);`, it's
> not dinamically linked at build time.
This is what I meant -
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #12 from Antonio T. (sagitter) ---
- libpasraw-1.3.0-3.fc33.src.rpm does not exist.
- %{forgesource} ??
- "make %{_smp_mflags} arch_flags="%{optflags}" LDFLAGS="%{build_ldflags}""
Better:
%build
%make_build
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #11 from Mattia Verga ---
Spec URL: https://mattia.fedorapeople.org/libpasraw.spec
SRPM URL: https://mattia.fedorapeople.org/libpasraw-1.3.0-3.fc33.src.rpm
Description: Provides shared library to interface Pascal program with
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #10 from Mattia Verga ---
(In reply to Fabio Valentini from comment #9)
> (In reply to Mattia Verga from comment #8)
> > I've asked on the packaging mailing list if this is really mandatory...
> > there are plenty of private
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
Fabio Valentini changed:
What|Removed |Added
CC||decatho...@gmail.com
--- Comment
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #8 from Mattia Verga ---
(In reply to Antonio T. (sagitter) from comment #7)
> (In reply to Mattia Verga from comment #5)
> >
> > > - Please, create a devel subpackage and include an symbolic link
> > > `libpasraw.so` pointed to
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #7 from Antonio T. (sagitter) ---
(In reply to Mattia Verga from comment #5)
>
> > - Please, create a devel subpackage and include an symbolic link
> > `libpasraw.so` pointed to `libpasraw.so.1.1`
> > The source code should
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #6 from Mattia Verga ---
Spec URL: https://mattia.fedorapeople.org/libpasraw.spec
SRPM URL: https://mattia.fedorapeople.org/libpasraw-1.3.0-2.fc33.src.rpm
Description: Provides shared library to interface Pascal program with
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #5 from Mattia Verga ---
Thanks for the review, but I don't agree with all points:
> - LICENSE file reports a GPLv3
I've reported upstream that LICENSE file (GPLv3+) is incoherent wit copyright
file (GPLv2+)
> -
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #4 from Antonio T. (sagitter) ---
(In reply to Antonio T. (sagitter) from comment #3)
>
> - This package provides a library earlier included in `libpasastro-1.2.*`; i
> guess it's better this way:
>
> new `libpasastro = 1.3.0-1`
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #3 from Antonio T. (sagitter) ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- LICENSE file reports a GPLv3
-
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
--- Comment #2 from Antonio T. (sagitter) ---
Created attachment 1670156
--> https://bugzilla.redhat.com/attachment.cgi?id=1670156=edit
ldflags patch
--
You are receiving this mail because:
You are on the CC list for the bug.
You are
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
Antonio T. (sagitter) changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
Mattia Verga changed:
What|Removed |Added
Blocks||1813566
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1813563
Mattia Verga changed:
What|Removed |Added
Doc Type|--- |If docs needed, set a value
---
20 matches
Mail list logo
