Beefy Boxes and Bandwidth Generously Provided by pair Networks
Do you know where your variables are?
 
PerlMonks  

Re: Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc

by Tux (Canon)
on Nov 18, 2011 at 11:53 UTC ( [id://938791]=note: print w/replies, xml ) Need Help??


in reply to Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc

Without looking at the content, I'll just have to say that PerlMonks is not the right place to propose these kind of changes. Those who know more about this are on the Perl5 Porters Mailing List. Feel free to post it there.


Enjoy, Have FUN! H.Merijn
  • Comment on Re: Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc

Replies are listed 'Best First'.
Re^2: Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc
by zengargoyle (Deacon) on Nov 18, 2011 at 12:44 UTC

    I have posted it (well, an earlier hackier version) to p5p. I'm looking more for things like: mayberelocate() is an awful function name. that's not how you use SvREFCNT_dec(). you should put your function above S_incpush_if_exists(). needs more comments.

    I'd like to improve my chances of my patch being accepted before submitting it via perlbug. It might get overlooked or just passed over for being too crude on p5p, and well, the more looking over it gets the better.

      Those are all laudable goals!


      Enjoy, Have FUN! H.Merijn
Re^2: Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc
by Anonymous Monk on Nov 18, 2011 at 11:57 UTC

    Without looking at the content, I'll just have to say that PerlMonks is not the right place to propose these kind of changes.

    Sure it is

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://938791]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others examining the Monastery: (2)
As of 2024-04-25 05:39 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found