[Bug 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
perl-Cwd-Guard-0.04-2.el7 has been pushed to the Fedora EPEL 7 stable
repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
perl-Software-License-CCpack-1.01-3.el7 has been pushed to the Fedora EPEL 7
stable repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #16 from Fedora Update System upda...@fedoraproject.org ---
perl-Cwd-Guard-0.04-2.fc20 has been pushed to the Fedora 20 stable repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
perl-Software-License-CCpack-1.01-3.fc20 has been pushed to the Fedora 20
stable repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #18 from Fedora Update System upda...@fedoraproject.org ---
perl-Cwd-Guard-0.04-2.fc19 has been pushed to the Fedora 19 stable repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Software-License-CCpac
   ||k-1.01-3.fc21
 Resolution|--- |ERRATA
Last Closed||2014-10-14 00:33:38



--- Comment #14 from Fedora Update System upda...@fedoraproject.org ---
perl-Software-License-CCpack-1.01-3.fc21 has been pushed to the Fedora 21
stable repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|perl-Software-License-CCpac |perl-Cwd-Guard-0.04-2.fc21
   |k-1.01-3.fc21   |



--- Comment #15 from Fedora Update System upda...@fedoraproject.org ---
perl-Cwd-Guard-0.04-2.fc21 has been pushed to the Fedora 21 stable repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
perl-Cwd-Guard-0.04-2.fc21 has been pushed to the Fedora 21 testing repository.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 Blocks||1150121




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1150121
[Bug 1150121] Review Request: perl-Module-Build-XSUtil - A Module::Build
class for building XS modules
-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.com
  Flags||fedora-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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #1 from Petr Šabata psab...@redhat.com ---
Hmm, the perl comments in your description won't show, as they also happen to
be SPEC comments.  Is there a way to escape them?

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #2 from Paul Howarth p...@city-fan.org ---
I've tried a few things, including macros, but nothing seems to work.

Could possibly use a sort of pseudo-code instead, e.g.:

  use Cwd::Guard qw/cwd_guard/;
  use Cwd;

  my $dir = getcwd;
  MYBLOCK: {
my $guard = cwd_guard('/tmp/x') or die
  failed chdir: $Cwd::Guard::Error;
 directory is now /tmp/x 
  }
   directory is now back to $dir 

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Petr Šabata psab...@redhat.com ---
They're visible if there's something else than whitespace before them, so the
following would also work:

  use Cwd::Guard qw/cwd_guard/;
  use Cwd;

  my $dir = getcwd;
  MYBLOCK: {
my $guard = cwd_guard('/tmp/x') or die
  failed chdir: $Cwd::Guard::Error; # chdir to /tmp/x
  } # back to $dir


Up to you :)
There were no other issues, so I'll approve this now.

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #4 from Paul Howarth p...@city-fan.org ---
New Package SCM Request
===
Package Name: perl-Cwd-Guard
Short Description: Temporarily change the current directory
Upstream URL: http://search.cpan.org/dist/Cwd-Guard/
Owners: pghmcfc
Branches: f19 f20 f21 epel7
InitialCC: perl-sig

Thanks for the review Petr. I'll go for yada yada yada operators:

  use Cwd::Guard qw/cwd_guard/;
  use Cwd;

  my $dir = getcwd;
  MYBLOCK: {
my $guard = cwd_guard('/tmp/x') or die
  failed chdir: $Cwd::Guard::Error;
... # chdir to /tmp/x
  }
  ... # back to $dir

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #5 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
perl-Software-License-CCpack-1.01-3.el7 has been submitted as an update for
Fedora EPEL 7.
https://admin.fedoraproject.org/updates/perl-Software-License-CCpack-1.01-3.el7

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
perl-Software-License-CCpack-1.01-3.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/perl-Software-License-CCpack-1.01-3.fc20

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
perl-Software-License-CCpack-1.01-3.fc21 has been submitted as an update for
Fedora 21.
https://admin.fedoraproject.org/updates/perl-Software-License-CCpack-1.01-3.fc21

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
perl-Cwd-Guard-0.04-2.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/perl-Cwd-Guard-0.04-2.el7

-- 
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 1150113] Review Request: perl-Cwd-Guard - Temporarily change the current directory

2014-10-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1150113



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
perl-Cwd-Guard-0.04-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/perl-Cwd-Guard-0.04-2.fc19

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