[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-09-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|4.92-3.fc11 |4.92-3.el5




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-09-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #25 from Fedora Update System   
2009-09-10 18:55:33 EDT ---
ipplan-4.92-3.el5 has been pushed to the Fedora EPEL 5 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-09-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|4.92-3.fc10 |4.92-3.fc11




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-09-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #24 from Fedora Update System   
2009-09-02 20:34:57 EDT ---
ipplan-4.92-3.fc11 has been pushed to the Fedora 11 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-09-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 510943 depends on bug 513636, which changed state.

Bug 513636 Summary: Review Request: php-layers-menu - Hierarchical PHP based 
DHTML menu system
https://bugzilla.redhat.com/show_bug.cgi?id=513636

   What|Old Value   |New Value

 Status|ON_QA   |CLOSED
 Resolution||ERRATA



--- Comment #23 from Fedora Update System   
2009-09-02 20:30:09 EDT ---
ipplan-4.92-3.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-09-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||4.92-3.fc10
 Resolution||ERRATA




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #21 from Fedora Update System   
2009-08-25 18:10:41 EDT ---
ipplan-4.92-3.fc11 has been pushed to the Fedora 11 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ipplan'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/F11/FEDORA-2009-8990

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #22 from Fedora Update System   
2009-08-25 18:12:57 EDT ---
ipplan-4.92-3.fc10 has been pushed to the Fedora 10 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ipplan'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/F10/FEDORA-2009-9010

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #20 from Fedora Update System   
2009-08-25 12:02:35 EDT ---
ipplan-4.92-3.el5 has been pushed to the Fedora EPEL 5 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ipplan'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/EL-5/FEDORA-EPEL-2009-0334

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #19 from Fedora Update System   
2009-08-25 05:28:24 EDT ---
ipplan-4.92-3.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/ipplan-4.92-3.el5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #18 from Fedora Update System   
2009-08-25 05:28:18 EDT ---
ipplan-4.92-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/ipplan-4.92-3.fc11

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #17 from Fedora Update System   
2009-08-25 05:28:12 EDT ---
ipplan-4.92-3.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/ipplan-4.92-3.fc10

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #16 from Kevin Fenzi   2009-08-24 15:40:34 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Andrew Colin Kissa  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #15 from Andrew Colin Kissa   2009-08-24 
04:20:09 EDT ---

David:

Thanks for the review and all the help.
- Fixed the requires.

New Package CVS Request
===
Package Name: ipplan
Short Description: Web-based IP address manager and tracker
Owners: topdog
Branches: F-10 F-11 EL-5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


David Nalley  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #14 from David Nalley   2009-08-23 19:36:55 EDT ---
Andrew: 

Oustanding 

I note that you still have requires for /sbin/service, though it's not used any
longer. I am sure you can fix that before it hits CVS. I have installed and am
playing with this package now. 
Thanks for your work on this! 

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from Andrew Colin Kissa   2009-08-23 
06:23:48 EDT ---

Thanks David,

* License Tag - fixed

* Bundled Library (php-PHPMailer) - unbundled

* License include - fixed

* Password generation - fixed

* Apache restart - fixed

- Updated spec and srpm
http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan.spec
http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan-4.92-2.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from David Nalley   2009-08-22 13:04:37 EDT ---
OK: rpmlint must be run on every package. The output should be posted in the
review.

[ke4...@nalleyt43 SPECS]$ rpmlint ./ipplan.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt43 SPECS]$ rpmlint ../SRPMS/ipplan-4.92-1.fc11.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt43 SPECS]$ rpmlint ../RPMS/noarch/ipplan-4.92-1.fc11.noarch.rpm 
ipplan.noarch: E: non-standard-dir-perm /var/cache/ipplan 0700
ipplan.noarch: E: non-readable /etc/ipplan/config.php 0640
1 packages and 0 specfiles checked; 2 errors, 0 warnings.




OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. 
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
GPLv2+ per source code

FIX: The License field in the package spec file must match the actual license.
SPEC file says GPLv2 should be GPLv2+ 

FIX: 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 must be included in %doc

gpl.html isn't pushed to %doc - I realize that this is likely due to the fact
that it is referenced by the application - but consider symlinking.

OK: The spec file must be written in American English.
OK: The spec file for the package MUST be legible. 
Ok: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines 

[ke4...@nalleyt43 SOURCES]$ md5sum ipplan-4.92.tar.gz*
2540b381744d1891b53b660fffc5ec56  ipplan-4.92.tar.gz
2540b381744d1891b53b660fffc5ec56  ipplan-4.92.tar.gz.1

OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. 
NA: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. 

FIX: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense. 

I think there is yet another bundled library - phpmailer 
http://sourceforge.net/projects/phpmailer/
This already exists in fedora as: php-PHPMailer

OK: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
NA: Every binary RPM package (or subpackage) which stores shared library files
(not just symlinks) in any of the dynamic linker's default paths, must call
ldconfig in %post and %postun. 
NA: If the package is designed to be relocatable, the packager must state this
fact in the request for review, along with the rationalization for relocation
of that specific package. Without this, use of Prefix: /usr is considered a
blocker. 
OK: A package must own all directories that it creates. If it does not create a
directory that it uses, then it should require a package which does create that
directory. 
OK: A Fedora package must not list a file more than once in the spec file's
%files listings. 
OK: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line. 
OK: Each package must have a %clean section, which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). 
OK: Each package must consistently use macros. 
OK: The package must contain code, or permissable content. 
NA: Large documentation files must go in a -doc subpackage. (The definition of
large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). 
NA: If a package includes something as %doc, it must not affect the runtime of
the application. To summarize: If it is in %doc, the program must run properly
if it is not present. 
NA: Header files must be in a -devel package. 
NA: Static libraries must be in a -static package. 
NA: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for
directory ownership and usability). 
NA: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suff

[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from David Nalley   2009-08-22 12:32:24 EDT ---
Andrew: 

sorry for my lag I will try and get this pushed out to you this weekend. 

Thanks for your work on this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Andrew Colin Kissa   2009-08-14 
06:27:37 EDT ---

Updated to upstream version 4.92

http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan.spec
http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan-4.92-1.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Andrew Colin Kissa   2009-08-13 
06:46:30 EDT ---

Gary,

I have submitted a request to maintain php-adodb for EPEL, I will work on
bumping ipplan to 4.92 soon.

If you would be so kind as to look at
https://bugzilla.redhat.com/show_bug.cgi?id=513954 if you can, which is holding
up php-layers-menu which in turn will hold up ipplan.

Thanks in advance

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Gary T. Giesen   2009-08-12 19:34:23 
EDT ---
One more thing,

I believe the SRPM will have to be rebuilt with:

%_source_filedigest_algorithm md5
%_binary_filedigest_algorithm md5

in ~/.rpmmacros in order for this to build properly for EL-5 (RHEL5 doesn't
support the new digest algorithms that Fedora 10/11 use).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Gary T. Giesen  changed:

   What|Removed |Added

 CC||gie...@snickers.org




--- Comment #7 from Gary T. Giesen   2009-08-12 19:06:23 
EDT ---
I too would like to see this packaged, specifically in EPEL. Unfortunately,
php-adodb does not have an EPEL branch right now, and the maintainer (abombard)
is not interested in maintaning it. So there's a couple of options:

1) I would be willing to maintain the EPEL branches of php-adodb to ensure that
ipplan gets packaged

2) If you're interested, you could maintain the php-adodb EPEL branch. 

Now, since my interest in php-adodb is only because of ipplan, I would prefer
option 2, but I will do option 1 if that ensures that ipplan gets packaged.

Let me know which you prefer. Also note that an updated package (4.92) is
available from upstream so you may want to consider packaging that instead.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Juha Tuomala  changed:

   What|Removed |Added

URL||http://iptrack.sourceforge.
   ||net/
 CC||t...@iki.fi
  Alias||ipplan




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Andrew Colin Kissa   2009-07-27 
13:14:05 EDT ---
Updated spec because of changes to dependent packages.

http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan.spec
http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan-4.91-3.a.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Andrew Colin Kissa  changed:

   What|Removed |Added

 Depends on||513636




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Andrew Colin Kissa   2009-07-24 
11:37:41 EDT ---

I have made the changes to use the fedora supplied adodb and i have packaged
php-layers-menu https://bugzilla.redhat.com/show_bug.cgi?id=513636 it is
awaiting review.

As for layout.class the upstream does not seem to exist any more. I have tested
and very thing works for me.

The updated spec and srpm 
http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan.spec
http://topdog-software.com/oss/SRPMS/fedora/ipplan/ipplan-4.91-2.a.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Andrew Colin Kissa   2009-07-23 
06:36:38 EDT ---
Actually it is modifications to existing classes, i am going to try it with the
fedora one and see. Will update when i have results.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from David Nalley   2009-07-23 06:32:33 EDT ---
I can empathize.
Are you sure that ipplan won't work with the adodb currently in Fedora?? 
(I haven't looked at all at what has changed - so that might be complete
non-sense. On the other hand if they have been adding additional classes,
perhaps those could be congealed ina php-pear-adodb-ipplan package)

There is always FPC to appeal to as well, I suppose.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Andrew Colin Kissa   2009-07-23 
06:21:19 EDT ---
I can handle the latter two, am not sure how upstream will respond to bumping
up their adodb though as the bundled seems more like a fork as they have not
bothered to keep up with the mainstream adodb.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from David Nalley   2009-07-23 06:13:11 EDT ---
I'll start with the really painful stuff: Bundled libraries. 

So you know about adodb - and you mentioned it has been modified in the spec. 
The solutions are really to have upstream push their patches to adodb's
upstream and included, or fork adodb (and package separately). The version
appears to be quite dated anyway, being v4.54, and 5 years old while current
release is v5.09. 

php-openlayers-menu is also bundled, as is layout.class 


The latter two are currently not in Fedora based on my quick search, so they'd
need to be packaged. 

I am really interested in seeing ipplan make it into Fedora, so if you get
either of the above packaged, let me know and I'll try and knock out reviews
for those as well.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-07-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


David Nalley  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||da...@gnsa.us
 AssignedTo|nob...@fedoraproject.org|da...@gnsa.us
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review