[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #32 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/andy-super-great-park

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #31 from MartinKG  ---
@Jeremy
many thanks for your assistance !

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273

Jeremy Newton  changed:

   What|Removed |Added

  Flags||fedora-review+



--- Comment #30 from Jeremy Newton  ---
Looks good, approved!

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #29 from MartinKG  ---
(In reply to Jeremy Newton from comment #28)
> (In reply to MartinKG from comment #27)
> > (In reply to Jeremy Newton from comment #26)
> > > [!]: License field in the package spec file matches the actual license.
> > >  Note: Checking patched sources after %prep for licenses. Licenses
> > >  found: "Apache (v2.0)", "*No copyright* CC by-sa", "GPL (v2 or
> > >  later)", "*No copyright* CC by-sa (v3.0)", "Unknown or generated".
> > >  1555 files have unknown license.
> > > > License is GPLv3 not GPLv3+. See LICENSE file
> > 
> > done
> > > 
> > > [x]: License file installed when any subpackage combination is installed.
> > > [!]: If the package is under multiple licenses, the licensing breakdown
> > >  must be documented in the spec.
> > > > You should really add a breakdown if possible
> > 
> > done
> 
> I think you misunderstood. There are two licenses provided: CCPL (CC by-sa
> v3.0) and GPL.
> 
> According you the readme, all code is GPL v3 (excluding data/common.scm,
> which is actually GPLv2+) and all multimedia is CC by-sa v3.0. You can
> change the breakdown to the following:
> 
> > # All code is GPLv3 except:
> > # asgp/data/common.scm is GPL (v2 or later)
> > # All multimedia (pictures, sounds, levels, etc.) is CC BY-SA V3.0

done

> As well, asgp/android can be deleted in prep as you shouldn't need these
> files for Fedora (thus you should also remove ASL 2.0).
> 

done, delete android and removed ASL 2.0 license


> Also please include the full licenses in the data file %license macro like
> so:
> 
> > %license asgp/LICENSE asgp/license/*

done

> Note the %license macro for the main package is not necessary, since the
> main package requires the data package, but you may include it if you so
> please.
> 
> > > [!]: Package must own all directories that it creates.
> > >  Note: Directories without known owners:
> > >  /usr/share/icons/hicolor/24x24, /usr/share/icons/hicolor/64x64/apps,
> > >  /usr/share/icons/hicolor/48x48/apps,
> > >  /usr/share/icons/hicolor/32x32/apps,
> > >  /usr/share/icons/hicolor/24x24/apps,
> > >  /usr/share/icons/hicolor/16x16/apps, /usr/share/icons/hicolor/16x16,
> > >  /usr/share/icons/hicolor/128x128/apps,
> > >  /usr/share/icons/hicolor/128x128, /usr/share/icons/hicolor/64x64,
> > >  /usr/share/icons/hicolor/48x48, /usr/share/icons/hicolor/32x32,
> > >  /usr/share/icons/hicolor, /usr/share/bear-factory
> > > > This can be fixed by adding the following to the main package:
> > > BuildRequires: hicolor-icon-theme
> > > Requires: hicolor-icon-theme
> > > > Which is something I missed when I was reviewing bear. Please fix this.
> > 
> > - add RR hicolor-icon-theme in both packages, I think BR is not required.
> 
> Indeed, the BR is not required, but it should silence the fedora-review
> warning. Adding only the RR is fine by me.
> 
> The license issue is the only thing that needs to be fixed. You don't have
> to upload another srpm if you don't want to, but please post the new spec
> with the suggested changes before I can approve.

all suggestions hopefully changed.

new spec file only:

Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/asgp.spec

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #28 from Jeremy Newton  ---
(In reply to MartinKG from comment #27)
> (In reply to Jeremy Newton from comment #26)
> > [!]: License field in the package spec file matches the actual license.
> >  Note: Checking patched sources after %prep for licenses. Licenses
> >  found: "Apache (v2.0)", "*No copyright* CC by-sa", "GPL (v2 or
> >  later)", "*No copyright* CC by-sa (v3.0)", "Unknown or generated".
> >  1555 files have unknown license.
> > > License is GPLv3 not GPLv3+. See LICENSE file
> 
> done
> > 
> > [x]: License file installed when any subpackage combination is installed.
> > [!]: If the package is under multiple licenses, the licensing breakdown
> >  must be documented in the spec.
> > > You should really add a breakdown if possible
> 
> done

I think you misunderstood. There are two licenses provided: CCPL (CC by-sa
v3.0) and GPL.

According you the readme, all code is GPL v3 (excluding data/common.scm, which
is actually GPLv2+) and all multimedia is CC by-sa v3.0. You can change the
breakdown to the following:

> # All code is GPLv3 except:
> # asgp/data/common.scm is GPL (v2 or later)
> # All multimedia (pictures, sounds, levels, etc.) is CC BY-SA V3.0

As well, asgp/android can be deleted in prep as you shouldn't need these files
for Fedora (thus you should also remove ASL 2.0).

Also please include the full licenses in the data file %license macro like so:

> %license asgp/LICENSE asgp/license/*

Note the %license macro for the main package is not necessary, since the main
package requires the data package, but you may include it if you so please.

> > [!]: Package must own all directories that it creates.
> >  Note: Directories without known owners:
> >  /usr/share/icons/hicolor/24x24, /usr/share/icons/hicolor/64x64/apps,
> >  /usr/share/icons/hicolor/48x48/apps,
> >  /usr/share/icons/hicolor/32x32/apps,
> >  /usr/share/icons/hicolor/24x24/apps,
> >  /usr/share/icons/hicolor/16x16/apps, /usr/share/icons/hicolor/16x16,
> >  /usr/share/icons/hicolor/128x128/apps,
> >  /usr/share/icons/hicolor/128x128, /usr/share/icons/hicolor/64x64,
> >  /usr/share/icons/hicolor/48x48, /usr/share/icons/hicolor/32x32,
> >  /usr/share/icons/hicolor, /usr/share/bear-factory
> > > This can be fixed by adding the following to the main package:
> > BuildRequires: hicolor-icon-theme
> > Requires: hicolor-icon-theme
> > > Which is something I missed when I was reviewing bear. Please fix this.
> 
> - add RR hicolor-icon-theme in both packages, I think BR is not required.

Indeed, the BR is not required, but it should silence the fedora-review
warning. Adding only the RR is fine by me.

The license issue is the only thing that needs to be fixed. You don't have to
upload another srpm if you don't want to, but please post the new spec with the
suggested changes before I can approve.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #27 from MartinKG  ---
(In reply to Jeremy Newton from comment #26)
> Package Review
> ==
> 
> Legend:
> [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
> 
> = MUST items =
> 
> C/C++:
> [x]: Package does not contain kernel modules.
> [x]: Package contains no static executables.
> [-]: Development (unversioned) .so files in -devel subpackage, if present.
>  Note: Unversioned so-files in private %_libdir subdirectory (see
>  attachment). Verify they are not in ld path.
> > This is fine, no need to change this.

ok
> 
> [x]: Package does not contain any libtool archives (.la)
> [x]: Rpath absent or only used for internal libs.
> 
> Generic:
> [x]: Package is licensed with an open-source compatible license and meets
>  other legal requirements as defined in the legal section of Packaging
>  Guidelines.
> [!]: License field in the package spec file matches the actual license.
>  Note: Checking patched sources after %prep for licenses. Licenses
>  found: "Apache (v2.0)", "*No copyright* CC by-sa", "GPL (v2 or
>  later)", "*No copyright* CC by-sa (v3.0)", "Unknown or generated".
>  1555 files have unknown license.
> > License is GPLv3 not GPLv3+. See LICENSE file

done
> 
> [x]: License file installed when any subpackage combination is installed.
> [!]: If the package is under multiple licenses, the licensing breakdown
>  must be documented in the spec.
> > You should really add a breakdown if possible

done
> 
> [!]: Package must own all directories that it creates.
>  Note: Directories without known owners:
>  /usr/share/icons/hicolor/24x24, /usr/share/icons/hicolor/64x64/apps,
>  /usr/share/icons/hicolor/48x48/apps,
>  /usr/share/icons/hicolor/32x32/apps,
>  /usr/share/icons/hicolor/24x24/apps,
>  /usr/share/icons/hicolor/16x16/apps, /usr/share/icons/hicolor/16x16,
>  /usr/share/icons/hicolor/128x128/apps,
>  /usr/share/icons/hicolor/128x128, /usr/share/icons/hicolor/64x64,
>  /usr/share/icons/hicolor/48x48, /usr/share/icons/hicolor/32x32,
>  /usr/share/icons/hicolor, /usr/share/bear-factory
> > This can be fixed by adding the following to the main package:
> BuildRequires: hicolor-icon-theme
> Requires: hicolor-icon-theme
> > Which is something I missed when I was reviewing bear. Please fix this.

- add RR hicolor-icon-theme in both packages, I think BR is not required.

> [x]: %build honors applicable compiler flags or justifies otherwise.
> [x]: Package contains no bundled libraries without FPC exception.
> [x]: Changelog in prescribed format.
> [x]: Sources contain only permissible code or content.
> [-]: Development files must be in a -devel package
> [x]: Package uses nothing in %doc for runtime.
> [x]: The spec file handles locales properly.
> [x]: Package consistently uses macros (instead of hard-coded directory
>  names).
> [x]: Package is named according to the Package Naming Guidelines.
> [x]: Package does not generate any conflict.
> [x]: Package obeys FHS, except libexecdir and /usr/target.
> [-]: If the package is a rename of another package, proper Obsoletes and
>  Provides are present.
> [x]: Requires correct, justified where necessary.
> [x]: Spec file is legible and written in American English.
> [-]: Package contains systemd file(s) if in need.
> [x]: gtk-update-icon-cache is invoked in %postun and %posttrans if package
>  contains icons.
>  Note: icons in asgp
> [x]: Useful -debuginfo package or justification otherwise.
> [x]: Package is not known to require an ExcludeArch tag.
> > ExcludeArch is required, but this is fine until bear is fixed on ppc64le... 
> > This arch is largely broken for a lot of stuff, so it's not a big deal.
ok

> 
> = SHOULD items =
> 
> Generic:
> [-]: Avoid bundling fonts in non-fonts packages.
>  Note: Package contains font files
> [-]: If the source package does not include license text(s) as a separate
>  file from upstream, the packager SHOULD query upstream to include it.
> [x]: Final provides and requires are sane (see attachments).
> [-]: Fully versioned dependency in subpackages if applicable.
>  Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in asgp-
>  data , asgp-debuginfo
> [?]: Package functions as described.
> [?]: Latest version is packaged.
> [x]: Package does not include license text files separate from upstream.
> [-]: Patches link to upstream bugs/comments/lists or are otherwise
>  justified.
> > If this patch is upstreamable, I would send it, elsewise it doesn't matter 
> > too much.

done

new rpms:
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/asgp.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/asgp-1.0.18-4.fc25.src.rpm

%changelog
* Mon Jan 02 2017 Martin Gansser  - 1.0.18-4
- correct license to GPLv3
- add license breakdown
- add BR  

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #26 from Jeremy Newton  ---
Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated

= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[-]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.
> This is fine, no need to change this.

[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Apache (v2.0)", "*No copyright* CC by-sa", "GPL (v2 or
 later)", "*No copyright* CC by-sa (v3.0)", "Unknown or generated".
 1555 files have unknown license.
> License is GPLv3 not GPLv3+. See LICENSE file

[x]: License file installed when any subpackage combination is installed.
[!]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
> You should really add a breakdown if possible

[!]: Package must own all directories that it creates.
 Note: Directories without known owners:
 /usr/share/icons/hicolor/24x24, /usr/share/icons/hicolor/64x64/apps,
 /usr/share/icons/hicolor/48x48/apps,
 /usr/share/icons/hicolor/32x32/apps,
 /usr/share/icons/hicolor/24x24/apps,
 /usr/share/icons/hicolor/16x16/apps, /usr/share/icons/hicolor/16x16,
 /usr/share/icons/hicolor/128x128/apps,
 /usr/share/icons/hicolor/128x128, /usr/share/icons/hicolor/64x64,
 /usr/share/icons/hicolor/48x48, /usr/share/icons/hicolor/32x32,
 /usr/share/icons/hicolor, /usr/share/bear-factory
> This can be fixed by adding the following to the main package:
BuildRequires: hicolor-icon-theme
Requires: hicolor-icon-theme
> Which is something I missed when I was reviewing bear. Please fix this.

[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: The spec file handles locales properly.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: gtk-update-icon-cache is invoked in %postun and %posttrans if package
 contains icons.
 Note: icons in asgp
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
> ExcludeArch is required, but this is fine until bear is fixed on ppc64le... 
> This arch is largely broken for a lot of stuff, so it's not a big deal.

[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install or
 desktop-file-validate if there is such a file.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are 

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273
Bug 1176273 depends on bug 1398949, which changed state.

Bug 1398949 Summary: Review Request: bear - Game engine and editors dedicated 
to creating great 2D games
https://bugzilla.redhat.com/show_bug.cgi?id=1398949

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE



-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #25 from MartinKG  ---
new rpms:

Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/asgp.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/asgp-1.0.18-3.fc25.src.rpm

%changelog
* Fri Dec 23 2016 Martin Gansser  - 1.0.18-3
- remove Source1
- convert docbook2man filename taken from .sgml file to lowercase
- add macor %%license

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #24 from Jeremy Newton  ---
Just from a quick glance. Source1 shouldn't be necessary, please remove it.

I have some additional comments, but I need to look into it further to give you
better feedback.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #23 from MartinKG  ---
asgp runs now, new rpm packages:

Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/asgp.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/asgp-1.0.18-2.fc25.src.rpm

%changelog
* Tue Dec 13 2016 Martin Gansser  - 1.0.18-2
- Add BR bear-devel
- Add BR cmake option -DBEAR_ROOT_DIRECTORY=%%{_includedir}/bear
- Remove bear Sources
- Add LINK_DIRECTORIES(/usr/lib64/bear/) to CMakeLists.txt
- Delete subdirs bear in CMakeLists.txt

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #22 from MartinKG  ---

Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/asgp.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/asgp-1.0.18-2.fc25.src.rpm

%changelog
* Tue Dec 13 2016 Martin Gansser  - 1.0.18-2
- Add BR bear-devel
- Add BR cmake option -DBEAR_ROOT_DIRECTORY=%%{_includedir}/bear
- Remove bear Sources
- Add LINK_DIRECTORIES(/usr/lib64/bear/) to CMakeLists.txt
- Delete subdirs bear in CMakeLists.txt

CMakeLists file looks now:

diff -Naur CMakeLists.txt.orig CMakeLists.txt
--- CMakeLists.txt.orig2016-12-13 21:25:53.137347804 +0100
+++ CMakeLists.txt2016-12-13 21:39:11.399075740 +0100
@@ -6,6 +6,8 @@

 set( CMAKE_MODULE_PATH "${BEAR_ROOT_DIRECTORY}/cmake-helper" )

+LINK_DIRECTORIES(/usr/lib64/bear/)
+
 include( uninstall )

-subdirs( bear asgp )
+subdirs( asgp )

asgp compiles fine, but when i run the program, i get this error message:

martin@fc25 ~]$ asgp 
Exception: /usr/include/bear/bear-engine/bin/libbear_generic_items.so: cannot
open shared object file: No such file or directory
asgp [-h] [--log-concise=integer] [--log-file=file] [--log-level=string]
[--log-uniq] engine_options

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #21 from Jeremy Newton  ---
(In reply to MartinKG from comment #20)
> I have only uploaded the spec file due my thin internet bandwidth.
> Spec URL:
> https://martinkg.fedorapeople.org/Review/asgp-1.0.18/test2/asgp.spec
> 
> there's a problem with asgp/CMakeLists.txt see below:
> 
> + /usr/bin/cmake -DCMAKE_C_FLAGS_RELEASE:STRING=-DNDEBUG
> -DCMAKE_CXX_FLAGS_RELEASE:STRING=-DNDEBUG
> -DCMAKE_Fortran_FLAGS_RELEASE:STRING=-DNDEBUG
> -DCMAKE_VERBOSE_MAKEFILE:BOOL=ON -DCMAKE_INSTALL_PREFIX:PATH=/usr
> -DINCLUDE_INSTALL_DIR:PATH=/usr/include -DLIB_INSTALL_DIR:PATH=/usr/lib64
> -DSYSCONF_INSTALL_DIR:PATH=/etc -DSHARE_INSTALL_PREFIX:PATH=/usr/share
> -DLIB_SUFFIX=64 -DBUILD_SHARED_LIBS:BOOL=ON
> -DBEAR_ENGINE_INSTALL_LIBRARY_DIR=lib64/bear -DBEAR_EDITORS_ENABLED=OFF
> -DRP_INSTALL_CUSTOM_LIBRARY_DIR=lib64/bear
> -DBEAR_ROOT_DIRECTORY=/usr/include/bear -DRP_INSTALL_DATA_DIR=share/asgp
> -DRP_BEAR_FACTORY_ENABLED=ON
> -- The C compiler identification is GNU 6.2.1
> -- The CXX compiler identification is GNU 6.2.1
> -- Check for working C compiler: /usr/bin/cc
> -- Check for working C compiler: /usr/bin/cc -- works
> -- Detecting C compiler ABI info
> -- Detecting C compiler ABI info - done
> -- Detecting C compile features
> -- Detecting C compile features - done
> -- Check for working CXX compiler: /usr/bin/c++
> -- Check for working CXX compiler: /usr/bin/c++ -- works
> -- Detecting CXX compiler ABI info
> -- Detecting CXX compiler ABI info - done
> -- Detecting CXX compile features
> -- Detecting CXX compile features - done
> CMake Error at CMakeLists.txt:9 (include):
>   include could not find load file:
> 
> uninstall
> 
> 
> CMake Error at CMakeLists.txt:11 (subdirs):
>   subdirs Incorrect SUBDIRS command.  Directory: bear does not exist.
> 
> 
> CMake Error at asgp/CMakeLists.txt:19 (include):
>   include could not find load file:
> 
> gettext
> 
> 
> CMake Error at asgp/CMakeLists.txt:20 (include):
>   include could not find load file:
> 
> compiler-defaults
> 
> 
> CMake Error at asgp/CMakeLists.txt:21 (include):
>   include could not find load file:
> 
> docbook-to-man
> 
> 
> CMake Error at asgp/CMakeLists.txt:58 (include):
>   include could not find load file:
> 
> FindSDL2
> 
> 
> CMake Error at asgp/CMakeLists.txt:61 (message):
>   SDL2 library must be installed.
> 
> 
> -- Configuring incomplete, errors occurred!

Indeed, this is due to missing files cmake files in the bear-devel package,
I'll post a fix over there.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #20 from MartinKG  ---
I have only uploaded the spec file due my thin internet bandwidth.
Spec URL: https://martinkg.fedorapeople.org/Review/asgp-1.0.18/test2/asgp.spec

there's a problem with asgp/CMakeLists.txt see below:

+ /usr/bin/cmake -DCMAKE_C_FLAGS_RELEASE:STRING=-DNDEBUG
-DCMAKE_CXX_FLAGS_RELEASE:STRING=-DNDEBUG
-DCMAKE_Fortran_FLAGS_RELEASE:STRING=-DNDEBUG -DCMAKE_VERBOSE_MAKEFILE:BOOL=ON
-DCMAKE_INSTALL_PREFIX:PATH=/usr -DINCLUDE_INSTALL_DIR:PATH=/usr/include
-DLIB_INSTALL_DIR:PATH=/usr/lib64 -DSYSCONF_INSTALL_DIR:PATH=/etc
-DSHARE_INSTALL_PREFIX:PATH=/usr/share -DLIB_SUFFIX=64
-DBUILD_SHARED_LIBS:BOOL=ON -DBEAR_ENGINE_INSTALL_LIBRARY_DIR=lib64/bear
-DBEAR_EDITORS_ENABLED=OFF -DRP_INSTALL_CUSTOM_LIBRARY_DIR=lib64/bear
-DBEAR_ROOT_DIRECTORY=/usr/include/bear -DRP_INSTALL_DATA_DIR=share/asgp
-DRP_BEAR_FACTORY_ENABLED=ON
-- The C compiler identification is GNU 6.2.1
-- The CXX compiler identification is GNU 6.2.1
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at CMakeLists.txt:9 (include):
  include could not find load file:

uninstall


CMake Error at CMakeLists.txt:11 (subdirs):
  subdirs Incorrect SUBDIRS command.  Directory: bear does not exist.


CMake Error at asgp/CMakeLists.txt:19 (include):
  include could not find load file:

gettext


CMake Error at asgp/CMakeLists.txt:20 (include):
  include could not find load file:

compiler-defaults


CMake Error at asgp/CMakeLists.txt:21 (include):
  include could not find load file:

docbook-to-man


CMake Error at asgp/CMakeLists.txt:58 (include):
  include could not find load file:

FindSDL2


CMake Error at asgp/CMakeLists.txt:61 (message):
  SDL2 library must be installed.


-- Configuring incomplete, errors occurred!

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #19 from Jeremy Newton  ---
I still get the claw error when building in mock.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #18 from MartinKG  ---
Spec URL: https://martinkg.fedorapeople.org/Review/asgp-1.0.18/test1/asgp.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/asgp-1.0.18/test1/asgp-1.0.18-1.fc25.src.rpm

%changelog
* Fri Dec 09 2016 Martin Gansser  - 1.0.18-1
- Update to 1.0.18
- Update Source URL
- Add Requires bear-engine >= %%{bear_version}
- Add Requires bear-factory >= %%{bear_version}

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #17 from Jeremy Newton  ---
(In reply to Jeremy Newton from comment #16)
> (In reply to MartinKG from comment #15)
> for bear, where longhash is the bear git commit hash:
> https://github.com/j-jorge/bear/archive/%{longhash}.tar.gz

In other words, use something similar to what you did with the bear package you
have in review. The sources should be valid URLs where possible.
Obviously CMakeLists.txt would be exempt from this.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #16 from Jeremy Newton  ---
(In reply to MartinKG from comment #15)
> @Jeremy,
> Now I have managed to build a running asgp, but with a bundled bear-engine
> package. Can you have a look at it.
> 
> Spec URL:
> https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp.spec
> SRPM URL:
> https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp-1.0.16-1.
> fc25.src.rpm
> 
> changelog
> * Tue Nov 22 2016 Martin Gansser  - 1.0.16-1
> - Update to 1.0.16
> - Update Source URL

I gave it a shot, but it seems cmake is having trouble finding libclaw (I'm
using f25 with mock):

CMake Error at /usr/share/cmake/libclaw/libclaw-config.cmake:377 (MESSAGE):
  Could not find Claw library

Also, why are you using git clone to get the sources, when you can just use the
two following download links?

for bear, where longhash is the bear git commit hash:
https://github.com/j-jorge/bear/archive/%{longhash}.tar.gz

and for asgp:
https://github.com/j-jorge/%{name}/archive/%{version}.tar.gz

As well, I've noticed that asgp 1.18 was released 3 days ago. I would encourage
you to target that if possible.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #15 from MartinKG  ---
@Jeremy,
Now I have managed to build a running asgp, but with a bundled bear-engine
package. Can you have a look at it.

Spec URL: https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp-1.0.16-1.fc25.src.rpm

changelog
* Tue Nov 22 2016 Martin Gansser  - 1.0.16-1
- Update to 1.0.16
- Update Source URL

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273

MartinKG  changed:

   What|Removed |Added

 Depends On||1398949



--- Comment #14 from MartinKG  ---
created bear package https://bugzilla.redhat.com/show_bug.cgi?id=1398949


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1398949
[Bug 1398949] Review Request: bear - Game engine and editors dedicated to
creating great 2D games
-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #13 from Jeremy Newton  ---
Actually, it looks like the "bear engine" (https://github.com/j-jorge/bear) is
being bundled with plee-the-bear. I'm not 100% familiar with the code, but it
looks like it should probably be made a separate package altogether so it can
be unbundled.

This would also make packaging "Tunnel" easier as well
(https://github.com/j-jorge/tunnel)

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #12 from Jeremy Newton  ---
Ok, so I've looked into it.

Looks like the maintainer for plee the bear has moved the library files to
%{_libdir}/plee-the-bear, so you'll have to add the following to cmake:

-DBEAR_ENGINE_LIBRARY_DIRECTORY=%{_libdir}/plee-the-bear

But this is problematic if this is a run-time dependency for asgp (is it?). I
would assume that one of the packagers for plee-the-bear should move this into
a "lib" sub package and make a devel package to provide the include files. It
seems the include files can be specified with:

-DBEAR_ENGINE_INCLUDE_DIRECTORY=LOCATION
and 
-DBEAR_ITEMS_INCLUDE_DIRECTORY=LOCATION

The packagers also removed the editor binaries, so if you would like those to
be included too, I would inquire to the maintainers to add them or ask to
become a co-maintainer for the package.

Either way, it looks like you'll need to talk to them to see how to move
forward. Feel free to add me as a CC if you email them or make another bug
report.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #11 from Jeremy Newton  ---
(In reply to MartinKG from comment #10)
> I still testing asgp to compile, but it fails with this error message:
> 
> [ 96%] Linking CXX shared library ../../../bin/librp.so
> cd /home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp/lib/src/rp && /usr/bin/cmake
> -E cmake_link_script CMakeFiles/rp.dir/link.txt --verbose=1
> /usr/bin/c++  -fPIC -O2 -g -pipe -Wall -Werror=format-security
> -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong
> --param=ssp-buffer-size=4 -grecord-gcc-switches
> -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic  -g 
> -Wl,-z,relro -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -shared
> -Wl,-soname,librp.so -o ../../../bin/librp.so
> CMakeFiles/rp.dir/code/add_ingame_layers.cpp.o
> CMakeFiles/rp.dir/code/attractable_item.cpp.o
> CMakeFiles/rp.dir/code/balloon.cpp.o CMakeFiles/rp.dir/code/bird.cpp.o
> CMakeFiles/rp.dir/code/bird_support.cpp.o CMakeFiles/rp.dir/code/bomb.cpp.o
> CMakeFiles/rp.dir/code/bonus.cpp.o CMakeFiles/rp.dir/code/boss.cpp.o
> CMakeFiles/rp.dir/code/boss_controller.cpp.o
> CMakeFiles/rp.dir/code/boss_teleport.cpp.o
> CMakeFiles/rp.dir/code/cable.cpp.o CMakeFiles/rp.dir/code/cannonball.cpp.o
> CMakeFiles/rp.dir/code/cart_controller.cpp.o
> CMakeFiles/rp.dir/code/cart.cpp.o CMakeFiles/rp.dir/code/config_file.cpp.o
> CMakeFiles/rp.dir/code/config_save.cpp.o CMakeFiles/rp.dir/code/crate.cpp.o
> CMakeFiles/rp.dir/code/cursor.cpp.o
> CMakeFiles/rp.dir/code/decorative_balloon.cpp.o
> CMakeFiles/rp.dir/code/end.cpp.o CMakeFiles/rp.dir/code/entity.cpp.o
> CMakeFiles/rp.dir/code/explosion.cpp.o CMakeFiles/rp.dir/code/game_key.cpp.o
> CMakeFiles/rp.dir/code/game_variables.cpp.o
> CMakeFiles/rp.dir/code/help_button.cpp.o CMakeFiles/rp.dir/code/hole.cpp.o
> CMakeFiles/rp.dir/code/http_request.cpp.o CMakeFiles/rp.dir/code/init.cpp.o
> CMakeFiles/rp.dir/code/interactive_item.cpp.o
> CMakeFiles/rp.dir/code/level_exit.cpp.o
> CMakeFiles/rp.dir/code/level_selector.cpp.o
> CMakeFiles/rp.dir/code/level_settings.cpp.o
> CMakeFiles/rp.dir/code/level_variables.cpp.o
> CMakeFiles/rp.dir/code/obstacle.cpp.o
> CMakeFiles/rp.dir/code/pause_game.cpp.o CMakeFiles/rp.dir/code/plank.cpp.o
> CMakeFiles/rp.dir/code/plunger.cpp.o
> CMakeFiles/rp.dir/code/serial_switcher.cpp.o
> CMakeFiles/rp.dir/code/show_key_layer.cpp.o
> CMakeFiles/rp.dir/code/switching.cpp.o CMakeFiles/rp.dir/code/tar.cpp.o
> CMakeFiles/rp.dir/code/tnt.cpp.o CMakeFiles/rp.dir/code/util.cpp.o
> CMakeFiles/rp.dir/code/wall.cpp.o CMakeFiles/rp.dir/code/zeppelin.cpp.o
> CMakeFiles/rp.dir/android/code/back_button_home_item.cpp.o
> CMakeFiles/rp.dir/android/code/java_activity.cpp.o
> CMakeFiles/rp.dir/layer/code/help_layer.cpp.o
> CMakeFiles/rp.dir/layer/code/key_layer.cpp.o
> CMakeFiles/rp.dir/layer/code/misc_layer.cpp.o
> CMakeFiles/rp.dir/layer/code/pause_layer.cpp.o
> CMakeFiles/rp.dir/layer/code/status_layer.cpp.o
> CMakeFiles/rp.dir/layer/status/code/background_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/balloon_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/bonus_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/boss_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/cannonball_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/lives_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/floating_score_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/plunger_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/score_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/status_component.cpp.o
> CMakeFiles/rp.dir/layer/status/code/time_component.cpp.o
> CMakeFiles/rp.dir/message/code/help_layer_starting_message.cpp.o
> CMakeFiles/rp.dir/message/code/key_layer_starting_message.cpp.o
> CMakeFiles/rp.dir/message/code/pause_message.cpp.o
> CMakeFiles/rp.dir/transition_effect/code/level_ending_effect.cpp.o
> CMakeFiles/rp.dir/transition_effect/code/level_starting_effect.cpp.o 
> -L/usr/lib64/libjpeg.so  -L/usr/lib64/libpng.so 
> -L/home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp/bin 
> -L/home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp/../bear/bear-engine/bin
> -lbear_engine -lbear_gui -lbear_generic_items -lclaw_configuration_file 
> /usr/bin/ld: cannot find -lbear_engine
> /usr/bin/ld: cannot find -lbear_gui
> /usr/bin/ld: cannot find -lbear_generic_items
> collect2: error: ld returned 1 exit status
> lib/src/rp/CMakeFiles/rp.dir/build.make:1840: recipe for target
> 'bin/librp.so' failed
> make[2]: *** [bin/librp.so] Error 1
> make[2]: Leaving directory '/home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp'
> CMakeFiles/Makefile2:254: recipe for target
> 'lib/src/rp/CMakeFiles/rp.dir/all' failed
> make[1]: *** [lib/src/rp/CMakeFiles/rp.dir/all] Error 2
> Makefile:130: recipe for target 'all' failed
> make: *** [all] Error 2
> 
> the build section looks like this:
> 
> %build
> cp %{SOURCE1} .
> cd asgp
> %cmake  -DCMAKE_BUILD_TYPE=release \
> -DCMAKE_SKIP_RPATH:BOOL=ON \

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #10 from MartinKG  ---
I still testing asgp to compile, but it fails with this error message:

[ 96%] Linking CXX shared library ../../../bin/librp.so
cd /home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp/lib/src/rp && /usr/bin/cmake -E
cmake_link_script CMakeFiles/rp.dir/link.txt --verbose=1
/usr/bin/c++  -fPIC -O2 -g -pipe -Wall -Werror=format-security
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong
--param=ssp-buffer-size=4 -grecord-gcc-switches
-specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -mtune=generic  -g 
-Wl,-z,relro -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -shared
-Wl,-soname,librp.so -o ../../../bin/librp.so
CMakeFiles/rp.dir/code/add_ingame_layers.cpp.o
CMakeFiles/rp.dir/code/attractable_item.cpp.o
CMakeFiles/rp.dir/code/balloon.cpp.o CMakeFiles/rp.dir/code/bird.cpp.o
CMakeFiles/rp.dir/code/bird_support.cpp.o CMakeFiles/rp.dir/code/bomb.cpp.o
CMakeFiles/rp.dir/code/bonus.cpp.o CMakeFiles/rp.dir/code/boss.cpp.o
CMakeFiles/rp.dir/code/boss_controller.cpp.o
CMakeFiles/rp.dir/code/boss_teleport.cpp.o CMakeFiles/rp.dir/code/cable.cpp.o
CMakeFiles/rp.dir/code/cannonball.cpp.o
CMakeFiles/rp.dir/code/cart_controller.cpp.o CMakeFiles/rp.dir/code/cart.cpp.o
CMakeFiles/rp.dir/code/config_file.cpp.o
CMakeFiles/rp.dir/code/config_save.cpp.o CMakeFiles/rp.dir/code/crate.cpp.o
CMakeFiles/rp.dir/code/cursor.cpp.o
CMakeFiles/rp.dir/code/decorative_balloon.cpp.o
CMakeFiles/rp.dir/code/end.cpp.o CMakeFiles/rp.dir/code/entity.cpp.o
CMakeFiles/rp.dir/code/explosion.cpp.o CMakeFiles/rp.dir/code/game_key.cpp.o
CMakeFiles/rp.dir/code/game_variables.cpp.o
CMakeFiles/rp.dir/code/help_button.cpp.o CMakeFiles/rp.dir/code/hole.cpp.o
CMakeFiles/rp.dir/code/http_request.cpp.o CMakeFiles/rp.dir/code/init.cpp.o
CMakeFiles/rp.dir/code/interactive_item.cpp.o
CMakeFiles/rp.dir/code/level_exit.cpp.o
CMakeFiles/rp.dir/code/level_selector.cpp.o
CMakeFiles/rp.dir/code/level_settings.cpp.o
CMakeFiles/rp.dir/code/level_variables.cpp.o
CMakeFiles/rp.dir/code/obstacle.cpp.o CMakeFiles/rp.dir/code/pause_game.cpp.o
CMakeFiles/rp.dir/code/plank.cpp.o CMakeFiles/rp.dir/code/plunger.cpp.o
CMakeFiles/rp.dir/code/serial_switcher.cpp.o
CMakeFiles/rp.dir/code/show_key_layer.cpp.o
CMakeFiles/rp.dir/code/switching.cpp.o CMakeFiles/rp.dir/code/tar.cpp.o
CMakeFiles/rp.dir/code/tnt.cpp.o CMakeFiles/rp.dir/code/util.cpp.o
CMakeFiles/rp.dir/code/wall.cpp.o CMakeFiles/rp.dir/code/zeppelin.cpp.o
CMakeFiles/rp.dir/android/code/back_button_home_item.cpp.o
CMakeFiles/rp.dir/android/code/java_activity.cpp.o
CMakeFiles/rp.dir/layer/code/help_layer.cpp.o
CMakeFiles/rp.dir/layer/code/key_layer.cpp.o
CMakeFiles/rp.dir/layer/code/misc_layer.cpp.o
CMakeFiles/rp.dir/layer/code/pause_layer.cpp.o
CMakeFiles/rp.dir/layer/code/status_layer.cpp.o
CMakeFiles/rp.dir/layer/status/code/background_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/balloon_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/bonus_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/boss_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/cannonball_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/lives_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/floating_score_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/plunger_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/score_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/status_component.cpp.o
CMakeFiles/rp.dir/layer/status/code/time_component.cpp.o
CMakeFiles/rp.dir/message/code/help_layer_starting_message.cpp.o
CMakeFiles/rp.dir/message/code/key_layer_starting_message.cpp.o
CMakeFiles/rp.dir/message/code/pause_message.cpp.o
CMakeFiles/rp.dir/transition_effect/code/level_ending_effect.cpp.o
CMakeFiles/rp.dir/transition_effect/code/level_starting_effect.cpp.o 
-L/usr/lib64/libjpeg.so  -L/usr/lib64/libpng.so 
-L/home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp/bin 
-L/home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp/../bear/bear-engine/bin
-lbear_engine -lbear_gui -lbear_generic_items -lclaw_configuration_file 
/usr/bin/ld: cannot find -lbear_engine
/usr/bin/ld: cannot find -lbear_gui
/usr/bin/ld: cannot find -lbear_generic_items
collect2: error: ld returned 1 exit status
lib/src/rp/CMakeFiles/rp.dir/build.make:1840: recipe for target 'bin/librp.so'
failed
make[2]: *** [bin/librp.so] Error 1
make[2]: Leaving directory '/home/martin/rpmbuild/BUILD/asgp-1.0.16/asgp'
CMakeFiles/Makefile2:254: recipe for target 'lib/src/rp/CMakeFiles/rp.dir/all'
failed
make[1]: *** [lib/src/rp/CMakeFiles/rp.dir/all] Error 2
Makefile:130: recipe for target 'all' failed
make: *** [all] Error 2

the build section looks like this:

%build
cp %{SOURCE1} .
cd asgp
%cmake  -DCMAKE_BUILD_TYPE=release \
-DCMAKE_SKIP_RPATH:BOOL=ON \
-DCMAKE_BUILD_TYPE=debug
%make_build

-- 
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

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #9 from Jeremy Newton  ---
More notes:

Please make use of the make macros as well:

make %{?_smp_mflags}
->
%make_build

make install DESTDIR=%{buildroot} INSTALL="install -p"
->
%make_install

As well, please clean up %cmake, as some of the parameters are already defined
by this macro. Specifically, the following should be removed:
-DCMAKE_VERBOSE_MAKEFILE=TRUE
-DCMAKE_C_FLAGS="-DNDEBUG %{optflags}"
-DCMAKE_CXX_FLAGS="-DNDEBUG %{optflags}"
-DCMAKE_BUILD_WITH_INSTALL_RPATH=OFF
-DCMAKE_INSTALL_PREFIX=%{_prefix}

Furthermore, I don't think these are necessary:
-DCMAKE_NO_BUILTIN_CHRPATH=ON
-DCMAKE_BUILD_WITH_INSTALL_RPATH=OFF
I think this is why you were having rpath issues, this should be ON not OFF:
-DCMAKE_SKIP_RPATH:BOOL=OFF
You can replace it with a macro instead, for example:
%cmake . %_cmake_skip_rpath

Note that the oname variable can be effectively removed with updates I
suggested in comment#8.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273

Jeremy Newton  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|alexjn...@gmail.com



--- Comment #8 from Jeremy Newton  ---
(In reply to MartinKG from comment #7)
> (In reply to Jeremy Newton from comment #6)
> > 
> > Any status on this? Are you still interested in packaging this?
> > I can review this if you're interested in a review swap.
> 
> ok lets us swap the review.

Sounds good, I have #1379798 available to review.

As for this review, please update the spec to the latest version (1.0.16) and
update the source url to something like this:
https://github.com/j-jorge/%{name}/archive/%{version}.tar.gz#/%{name}_%{version}.tar.gz

As well, please unbundle plee-the-bear if possible.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #7 from MartinKG  ---
(In reply to Jeremy Newton from comment #6)
> 
> Any status on this? Are you still interested in packaging this?
> I can review this if you're interested in a review swap.

ok lets us swap the review.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273

Jeremy Newton  changed:

   What|Removed |Added

 CC||alexjn...@gmail.com



--- Comment #6 from Jeremy Newton  ---
(In reply to MartinKG from comment #5)
> (In reply to Raphael Groner from comment #3)
> >
> > * SHOULD what is "Plee the Bear"? How is it related to this package? You
> > won't be able to create two individual subpackges for that 'bear' and
> > 'bear-factory' stuff. So consider to package that separately and unbundle.
> > https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
> > > of the Bear Engine for Plee the Bear & Andy's Super Great Park.
> > 
> 
> there is already a package plee-the-bear for Fedora and only needs to be
> unbundled.
> https://apps.fedoraproject.org/packages/plee-the-bear

Any status on this? Are you still interested in packaging this?
I can review this if you're interested in a review swap.

> yes this is a valid link for asgp-1.0.8
> 
> but there is a recent version 1.0.12 on https://github.com/j-jorge/asgp/
> build instruction:
> 
> mkdir asgp-build
> cd asgp-build
> git clone https://github.com/j-jorge/bear.git
> git clone https://github.com/j-jorge/asgp.git
> 
> [root@fc21 asgp-build]# du -sk *
> 1140264   asgp
> 37616 bear
> 
> approx. 1,17 GByte
> 
> should we use this version ?
There is a new version available. Also your URL comment doesn't correctly
explain how to get the sources, but fortunately enough, it should be sufficient
to update the URL to the following:
https://github.com/j-jorge/%{name}/archive/%{version}.tar.gz#/%{name}_%{version}.tar.gz

At a quick glance, a few things will need to be updated (for example, "%setup
-qn %{oname}-%{version}" can be replaced with "%setup -q"), but it's pretty
minor.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273

Ben Rosser  changed:

   What|Removed |Added

 Blocks||1364745




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1364745
[Bug 1364745] (FE-GAMESIG) Games SIG tracker bug
-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #5 from MartinKG mgans...@alice.de ---
(In reply to Raphael Groner from comment #3)

 * SHOULD what is Plee the Bear? How is it related to this package? You
 won't be able to create two individual subpackges for that 'bear' and
 'bear-factory' stuff. So consider to package that separately and unbundle.
 https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
  of the Bear Engine for Plee the Bear  Andy's Super Great Park.
 

there is already a package plee-the-bear for Fedora and only needs to be
unbundled.
https://apps.fedoraproject.org/packages/plee-the-bear

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at
Summary|Review Request: |Review Request:
   |andy-super-great-park - |andy-super-great-park - 2D
   |andy-super-great-park   |arcade game



-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #2 from Volker Fröhlich volke...@gmx.at ---
Please make the build verbose and make Source0 a URL.

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273

Raphael Groner projects...@smart.ms changed:

   What|Removed |Added

 CC||projects...@smart.ms



--- Comment #3 from Raphael Groner projects...@smart.ms ---
Hi Martin,

there are some hints for your spec file. Maybe I can do the official review as
well.

* SHOULD consider to name the package 'asqp' as upstream does partly for the
project name? Well, I can see that the source tarball is using that full name
like in the subject.
https://fedoraproject.org/wiki/Packaging:NamingGuidelines#General_Naming

* SHOULD remove the Group tag cause it's obosolete. Note: This tag is
deprecated since Fedora 17.
https://fedoraproject.org/wiki/How_to_create_an_RPM_package#SPEC_file_overview

* MUST use a valid Source URL, or add a comment how to build the tarball.
https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/SourceURL

This?
http://www.stuff-o-matic.com/asgp/download/download.php?platform=source


* MUST build with mock and koji (scratch) to ensure all BR are correct.
https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRequires
https://fedoraproject.org/wiki/Using_the_Koji_build_system#Scratch_Builds

* SHOULD fix/patch CMakeLists.txt to not enforce docbook2x as it seems to
create conflicts. And please send your patch to upstream, do they know about
that issue?
https://fedoraproject.org/wiki/Packaging:Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment
 # Cmake suggests it but parser error will be got.
 BuildConflicts: docbook2x

* SHOULD what is Plee the Bear? How is it related to this package? You won't
be able to create two individual subpackges for that 'bear' and 'bear-factory'
stuff. So consider to package that separately and unbundle.
https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
 of the Bear Engine for Plee the Bear  Andy's Super Great Park.

* MUST 'Require: hicolor-icon-theme' cause of the folder ownership.
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#File_and_Directory_Ownership
 %files
 …
 %{_datadir}/icons/hicolor/*/apps/asgp.png

* MUST (when possible) use the %cmake macro to avoid relisting of all those
parameters like RPATH etc.
https://fedoraproject.org/wiki/Packaging:Cmake

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1176273



--- Comment #4 from MartinKG mgans...@alice.de ---
Hi Raphael,
thanks for your reply,

(In reply to Raphael Groner from comment #3)
 Hi Martin,
 
 there are some hints for your spec file. Maybe I can do the official review
 as well.
 
 * SHOULD consider to name the package 'asqp' as upstream does partly for the
 project name? Well, I can see that the source tarball is using that full
 name like in the subject.
 https://fedoraproject.org/wiki/Packaging:NamingGuidelines#General_Naming

ok, i will name the package/spec file asgp/asgp.spec

 * SHOULD remove the Group tag cause it's obosolete. Note: This tag is
 deprecated since Fedora 17.
 https://fedoraproject.org/wiki/
 How_to_create_an_RPM_package#SPEC_file_overview

done

 * MUST use a valid Source URL, or add a comment how to build the tarball.
 https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/SourceURL
 
 This?
 http://www.stuff-o-matic.com/asgp/download/download.php?platform=source
 
yes this is a valid link for asgp-1.0.8

but there is a recent version 1.0.12 on https://github.com/j-jorge/asgp/
build instruction:

mkdir asgp-build
cd asgp-build
git clone https://github.com/j-jorge/bear.git
git clone https://github.com/j-jorge/asgp.git

[root@fc21 asgp-build]# du -sk *
1140264asgp
37616bear

approx. 1,17 GByte

should we use this version ?

 
 * MUST build with mock and koji (scratch) to ensure all BR are correct.
 https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRequires
 https://fedoraproject.org/wiki/Using_the_Koji_build_system#Scratch_Builds
 
already done, but no additional build requires are detected.

 * SHOULD fix/patch CMakeLists.txt to not enforce docbook2x as it seems to
 create conflicts. And please send your patch to upstream, do they know about
 that issue?
 https://fedoraproject.org/wiki/Packaging:
 Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment
  # Cmake suggests it but parser error will be got.
  BuildConflicts: docbook2x
 
can you send me a patch ?

 * SHOULD what is Plee the Bear? How is it related to this package? You
 won't be able to create two individual subpackges for that 'bear' and
 'bear-factory' stuff. So consider to package that separately and unbundle.
 https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
  of the Bear Engine for Plee the Bear  Andy's Super Great Park.
 
not done, because not clear.

 * MUST 'Require: hicolor-icon-theme' cause of the folder ownership.
 https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/
 Guidelines#File_and_Directory_Ownership
  %files
  …
  %{_datadir}/icons/hicolor/*/apps/asgp.png
 
added Require: hicolor-icon-theme
is this ok ?

 * MUST (when possible) use the %cmake macro to avoid relisting of all those
 parameters like RPATH etc.
 https://fedoraproject.org/wiki/Packaging:Cmake
- done, replaced cmake with %cmake

Spec URL: https://martinkg.fedorapeople.org/Review/asgp-1.0.8-3/asgp.spec

%changelog
* Sat Dec 20 2014 Martin Gansser marti...@fedoraproject.org - 1.0.8-3
- changed package name from andy-super-great-park to asgp
- added valid Source URL
- added download instruction
- removed group tag
- added CMAKE_VERBOSE_MAKEFILE=TRUE to %%make build verbose
- added Require hicolor-icon-theme
- used macro %%cmake

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review