[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2023-10-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Fedora Review Service  changed:

   What|Removed |Added

URL||http://pilotlog.stansoft.or
   ||g



--- Comment #14 from Fedora Review Service 
 ---
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6565818
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2117046-pilotlog/fedora-rawhide-x86_64/06565818-pilotlog/fedora-review/review.txt

Please take a look if any issues were found.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Report this comment as SPAM: 
https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202117046%23c14
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2023-10-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #13 from Stansoft  ---
Spec URL:
https://download.copr.fedorainfracloud.org/results/stansoft/pilotlog/fedora-rawhide-x86_64/06565804-pilotlog/pilotlog.spec
SRPM URL:
https://download.copr.fedorainfracloud.org/results/stansoft/pilotlog/fedora-rawhide-x86_64/06565804-pilotlog/pilotlog-10-1.fc40.src.rpm


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Report this comment as SPAM: 
https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202117046%23c13
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2023-10-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Stansoft  changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2023-09-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #12 from Stansoft  ---
Spec URL:
https://download.copr.fedorainfracloud.org/results/stansoft/pilotlog/fedora-rawhide-x86_64/06439446-pilotlog/pilotlog.spec
SRPM URL:
https://download.copr.fedorainfracloud.org/results/stansoft/pilotlog/fedora-rawhide-x86_64/06439446-pilotlog/pilotlog-10-1.fc40.src.rpm

unbundled aubit4gl, now using the Fedora repo package


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Report this comment as SPAM: 
https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202117046%23c12
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2023-08-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Stansoft  changed:

   What|Removed |Added

  Flags|needinfo- needinfo- |




-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2023-08-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Stansoft  changed:

   What|Removed |Added

  Flags|needinfo?(r...@stansoft.org) |needinfo- needinfo-




-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


needinfo denied: [Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2023-08-14 Thread bugzilla


Product: Fedora
Version: rawhide
Component: Package Review

Stansoft  has denied Package Review
's request for Stansoft
's needinfo:
Bug 2117046: Review Request: pilotlog -  A pilot logbook for logging flight
time
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #10 from Stansoft  ---
Spec URL: https://download.stansoft.org/files/pilotlog.spec
SRPM URL: https://download.stansoft.org/files/pilotlog-9-2.fc36.src.rpm

Added %make_install


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #9 from Stansoft  ---
(In reply to Jonathan Wright from comment #8)
> You need a `BuildRequires: gcc`. [1]

Added

> 
> > Requires(pre):  shadow-utils
> 
> Is this needed?  I removed it and everything compiled fine it seems.
> 

It is not needed, I have removed it.

> > I am packaging Postgresql for version control since the pilotlog program is
> > compiled against it. Is it a requirement that I use the Fedora postgresql
> > package and libs?
> 
> Unless there's a really good reason not to, yes.  The fact that you're
> trying to populate a database here is partially what I'm unsure about.  I'm
> not totally sure what the packaging policy is surrounding situations like
> this since Postgres could be used for other things already running, etc.
> 

Yes a database needs to be created and populated prior to compiling since the
programs need the database schema to compile.

> > %configure works but it does not compile when using %make_build
> 
> It doesn't seem to like compiling with multiple threads.  You can use
> `%make_build -j1` to force it to 1 thread but ideally you can figure out why
> it won't compile using multiple threads.  I believe you're upstream/the
> developer of this aren't you?

Yes I am the developer. I put `%make_build -j1` in for now, but I will figure
out why it doesn't work with multiple threads.

> 
> How are you test building the RPM?  You need to be using mock (or something
> like `fedpkg --release rawhide mockbuild`).
> 

rpmbuild --target x86_64 -ba pilotlog.spec


> Why not also use `%make_install` instead of copying everything into place by
> hand in the spec?  I see the Makefile has some installation components in it
> already.

The Makefile copies into /usr/local/pilotlog and I want to use the same
Makefile
for Fedora packaging that I am using for anyone that downloads the source
tarball
and compiles.

> 
> > Requires:   ncurses%{?_isa} >= 6.2
> 
> This should instead be `BuildRequires: ncurses-devel`.  RPM will then
> automatically generate the proper `Requires` since it builds against it. [2]
> 

I made this change.

> I think a bunch of your psql related stuff is failing.  I see a ton of this
> in the build logs:
> 
> 2022-08-11 18:29:25.788 CDT [570] ERROR:  relation "syscolatt" does not
> exist at character 81
> 2022-08-11 18:29:25.788 CDT [570] STATEMENT:   DECLARE chkscatt_0 CURSOR
> WITH HOLD FOR select syscolatt.def_format,seqno from syscolatt WHERE
> tabname='log_entry' AND colname='entry_date' ORDER BY seqno
> 
> ---
> 
> On further looking at the source tarball I notice that Postgres you're
> providing is already compiled which is definitely forbidden.  [3]
> 

I will make the change to use the Fedora postgresql.

> ---
> 
> 1.
> https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/
> #_buildrequires_and_requires
> 2.
> https://docs.fedoraproject.org/en-US/packaging-guidelines/
> #_package_dependencies
> 3.
> https://docs.fedoraproject.org/en-US/packaging-guidelines/what-can-be-
> packaged/#prebuilt-binaries-or-libraries


Spec URL: https://download.stansoft.org/files/pilotlog.spec
SRPM URL: https://download.stansoft.org/files/pilotlog-9-2.fc36.src.rpm


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #8 from Jonathan Wright  ---
You need a `BuildRequires: gcc`. [1]

> Requires(pre):  shadow-utils

Is this needed?  I removed it and everything compiled fine it seems.

> I am packaging Postgresql for version control since the pilotlog program is
> compiled against it. Is it a requirement that I use the Fedora postgresql
> package and libs?

Unless there's a really good reason not to, yes.  The fact that you're trying
to populate a database here is partially what I'm unsure about.  I'm not
totally sure what the packaging policy is surrounding situations like this
since Postgres could be used for other things already running, etc.

> %configure works but it does not compile when using %make_build

It doesn't seem to like compiling with multiple threads.  You can use
`%make_build -j1` to force it to 1 thread but ideally you can figure out why it
won't compile using multiple threads.  I believe you're upstream/the developer
of this aren't you?

How are you test building the RPM?  You need to be using mock (or something
like `fedpkg --release rawhide mockbuild`).

Why not also use `%make_install` instead of copying everything into place by
hand in the spec?  I see the Makefile has some installation components in it
already.

> Requires:   ncurses%{?_isa} >= 6.2

This should instead be `BuildRequires: ncurses-devel`.  RPM will then
automatically generate the proper `Requires` since it builds against it. [2]

I think a bunch of your psql related stuff is failing.  I see a ton of this in
the build logs:

2022-08-11 18:29:25.788 CDT [570] ERROR:  relation "syscolatt" does not exist
at character 81
2022-08-11 18:29:25.788 CDT [570] STATEMENT:   DECLARE chkscatt_0 CURSOR WITH
HOLD FOR select syscolatt.def_format,seqno from syscolatt WHERE
tabname='log_entry' AND colname='entry_date' ORDER BY seqno

---

On further looking at the source tarball I notice that Postgres you're
providing is already compiled which is definitely forbidden.  [3]

---

1.
https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/#_buildrequires_and_requires
2.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_package_dependencies
3.
https://docs.fedoraproject.org/en-US/packaging-guidelines/what-can-be-packaged/#prebuilt-binaries-or-libraries


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #7 from Stansoft  ---
(In reply to Jonathan Wright from comment #6)
> > # Omit /lib/.build-id to avoid conflict with included libs
> > %global _build_id_links none
> > # Filter internal libraries
> > %global _privatelibs ^%{_libdir}/%{name}/.*\\.so$
> > %global __provides_exclude ^(%{_privatelibs})$
> > %global __requires_exclude ^(%{_privatelibs})$
> > %global __requires_exclude_from ^(%{_privatelibs})$
> > %global __filter_GLIBC_PRIVATE 1
> 
> Since I can't get the source code I'm not sure what all of this is for but
> are you packaging libraries and trying to avoid using system libs?  You
> should almost always use system libs. [2][3]

I am packaging Postgresql for version control since the pilotlog program is
compiled against it. Is it a requirement that I use the Fedora postgresql
package and libs?

> 
> > Source0:%{name}-v%{version}-src.tar.bz2
> 
> This needs to be a URL. [1]
> 
> > ./configure
> > make
> 

%configure works but it does not compile when using %make_build

> You should use macros %configure and %make_build. [4]
> 
> > cp installpl updatepl pl changelog COPYING README \
> >   %{buildroot}%{_libdir}/%{name}
> 
> > %pre
> > %post
> > %preun
> > %postun
> 
> These should be removed.
> 
> > BuildRequires:  systemd-rpm-macros
> 
> Is this needed?
> 
> You probably don't need changelog, COPYING, and README here.  README and
> changelog should fall under %doc and COPYING under %license.  This takes
> care of copying them into the correct places for you. [5]
> 
> 1. https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/
> 2.
> https://docs.fedoraproject.org/en-US/packaging-guidelines/
> AutoProvidesAndRequiresFiltering/#_summary
> 3. https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling
> 4. https://docs.fedoraproject.org/en-US/packaging-guidelines/#_parallel_make
> 5. https://docs.fedoraproject.org/en-US/packaging-guidelines/#_documentation
> 
> There are some other parts of this I'm not sure about at all and we'll have
> to seek guidance from others.

Here are the updated spec and source rpm

Spec URL: https://download.stansoft.org/files/pilotlog.spec
SRPM URL: https://download.stansoft.org/files/pilotlog-9-2.fc36.src.rpm


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #6 from Jonathan Wright  ---
> # Omit /lib/.build-id to avoid conflict with included libs
> %global _build_id_links none
> # Filter internal libraries
> %global _privatelibs ^%{_libdir}/%{name}/.*\\.so$
> %global __provides_exclude ^(%{_privatelibs})$
> %global __requires_exclude ^(%{_privatelibs})$
> %global __requires_exclude_from ^(%{_privatelibs})$
> %global __filter_GLIBC_PRIVATE 1

Since I can't get the source code I'm not sure what all of this is for but are
you packaging libraries and trying to avoid using system libs?  You should
almost always use system libs. [2][3]

> Source0:%{name}-v%{version}-src.tar.bz2

This needs to be a URL. [1]

> ./configure
> make

You should use macros %configure and %make_build. [4]

> cp installpl updatepl pl changelog COPYING README \
>   %{buildroot}%{_libdir}/%{name}

> %pre
> %post
> %preun
> %postun

These should be removed.

> BuildRequires:  systemd-rpm-macros

Is this needed?

You probably don't need changelog, COPYING, and README here.  README and
changelog should fall under %doc and COPYING under %license.  This takes care
of copying them into the correct places for you. [5]

1. https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/
2.
https://docs.fedoraproject.org/en-US/packaging-guidelines/AutoProvidesAndRequiresFiltering/#_summary
3. https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling
4. https://docs.fedoraproject.org/en-US/packaging-guidelines/#_parallel_make
5. https://docs.fedoraproject.org/en-US/packaging-guidelines/#_documentation

There are some other parts of this I'm not sure about at all and we'll have to
seek guidance from others.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #5 from Stansoft  ---
Spec URL: https://download.stansoft.org/files/pilotlog.spec
SRPM URL: https://download.stansoft.org/files/pilotlog-9-1.fc36.src.rpm

Those were suppose to be direct links on sourceforge, but I see they do not
work either so use the above links instead.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #4 from Jonathan Wright  ---
Those URLs still aren't going to work.  The URLs need to be a direct link to
the files without a landing page, etc.

I've flagged your review as needing a sponsor but we can still get the review
knocked out.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046



--- Comment #3 from Stansoft  ---
Spec URL:
https://downloads.sourceforge.net/project/pilotlog/SRPMS/pilotlog.spec?ts=gABi9XEIsNWudsCZjlnxrR3Lsdu7gksJ90uYqjKTLJRvUeKHIqOv6Y6c3aW5k7OpaUFFcrRrCE_EW95wiFK8JUSdGHSCtA%3D%3D&r=https%3A%2F%2Fsourceforge.net%2Fprojects%2Fpilotlog%2Ffiles%2FSRPMS%2Fpilotlog.spec%2Fdownload
SRPM URL:
https://downloads.sourceforge.net/project/pilotlog/SRPMS/pilotlog-9-1.fc36.src.rpm?ts=gABi9W4aKnXYnUJyAr0fibYfRqDFEJV2DEmcB6w2d99DwYMP5WEuBrBSgAwtyJhFMRrbIQoW0LLr1vjd09Kf08BPhuCs5Q%3D%3D&r=https%3A%2F%2Fsourceforge.net%2Fprojects%2Fpilotlog%2Ffiles%2FSRPMS%2Fpilotlog-9-1.fc36.src.rpm%2Fdownload

I am not a Fedora packager so I will need a sponsor.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Jonathan Wright  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)



--- Comment #2 from Jonathan Wright  ---
Are you a Fedora packager?  Your FAS name isn't in the packager group.

https://accounts.fedoraproject.org/user/stansoft/



Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Jonathan Wright  changed:

   What|Removed |Added

   Doc Type|--- |If docs needed, set a value
 CC||jonat...@almalinux.org



--- Comment #1 from Jonathan Wright  ---
Spec URL and SRPM URL need to be direct links so fedora-review can download
them.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 2117046] Review Request: pilotlog - A pilot logbook for logging flight time

2022-08-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2117046

Stansoft  changed:

   What|Removed |Added

Comment|0   |updated



--- Comment #0 has been edited ---

Spec URL:
https://sourceforge.net/projects/pilotlog/files/SRPMS/pilotlog.spec/download
SRPM URL:
https://sourceforge.net/projects/pilotlog/files/SRPMS/pilotlog-9-1.fc36.src.rpm/download
Description: Pilot Log is a pilot logbook for logging flight time and
calculating aircraft weight & balance.
Fedora Account System Username: stansoft

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2117046
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-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/package-review@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue