bugmake - Bugs: bug #23273, Please include xkcd easteregg

 
 

bug #23273: Please include xkcd easteregg

Submitter:  None
Submitted:  Sat 17 May 2008 10:09:22 AM UTC
Votes: 1
 
Severity:  3 - Normal Item Group:  Enhancement
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open
Component Version:  None Operating System:  Any
Fixed Release:  None Triage Status:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

  Spam posted by anonymous
Fri 19 Dec 2008 08:20:28 PM UTC, comment #8: 

I just want to add that I did try it after reading the comic, and I would definitely have been wearing a smile on my face for the rest of the day had I seen it actually work like that.

Anonymous
Sat 01 Nov 2008 11:40:39 AM UTC, comment #7: 

Yes, I fully agree. I’m just curious if Paul likes it :-)

Anonymous
Fri 31 Oct 2008 11:19:12 PM UTC, comment #6: 

I think this suggestion is counter to the nature of easter-eggs.  An easter-egg is something the maintainer of a piece of software puts in because they want to.  IMO, if Paul thinks it's funny, he should do it.   If he doesn't, or thinks it's a marginal joke, then he should feel free to ignore what everyone else thinks and not do it.  Maintainers carry the burden of answering questions and digging up fixes when no one else cares; the fringe benefit is that decisions about easter-eggs should be solely theirs to make.

Anonymous
Fri 31 Oct 2008 09:30:25 PM UTC, comment #5: 

Any news on this? Does it have a chance?

Anonymous
Thu 10 Jul 2008 11:55:54 AM UTC, comment #4: 

patch attached

Posted this already on make mailing list, sorry for duplicate info, was not sure about adding patch here.

As simple as possible I think.
Don't call to fatal so that easter egg error text is clear of adornment.
Exits using die(2) as fatal does.
Doesn't trigger action if -k (continue on error) is specified.
There's already a strcmp() call in remake.c (strncmp used for this patch).
There's already a getuid() call in misc.c, I think for all platforms.
Tested on linux, solaris and cygwin.

$ ./make the cat some tea
make: * No rule to make target `the'.  Stop.

$ ./make me a sandwich
What? Make it yourself.

$ sudo ./make me a sandwich
Okay.

$ ./make -k me a sandwich
make: * No rule to make target `me'.
make: * No rule to make target `a'.
make: * No rule to make target `sandwich'.

$ ./make -f Makefile.me me a sandwich
make: Nothing to be done for `me'.
make: * No rule to make target `a'.  Stop.

$ ./make me a cup of tea please
What? Make it yourself.


(file #16091)

James Coleman <jamesc>
Thu 22 May 2008 09:24:10 PM UTC, comment #3: 

Sure, I know how to make this happen on my machine (also by shell aliases etc.), but that’s not the point of an easter egg. I imagine someone reading that comic strip, trying it out just for the sake of it and then be really surprised that it works.

Joachim Breitner <nomeata>
Thu 22 May 2008 09:18:55 PM UTC, comment #2: 

No need to patch make itself, just include something like the following in the desired Makefiles:

sandwich:
        @if test `whoami` = "root" ; then \
          $(ECHO) "Okay." ; \
        else \
          $(ECHO) "What? Make it yourself." ; \
        fi

David Fang <fangism>
Sat 17 May 2008 11:32:58 PM UTC, comment #1: 

+1 funny.

Dave Korn <davek>
Sat 17 May 2008 10:09:22 AM UTC, original submission:  

Hi,

this might be an odd request, but please consider it nevertheless.

There is a very popular web comic involving the make command:
http://xkcd.com/149/

It would not be useful, but entertaining, if the make command  would behave like this. So I’m proposing the following behaviour:

If make is run with the arguments "me a sandwich", and there is no target called "me", the error message is replaced by "What? Make it yourself.", if SUDO_COMMAND is unset, or "Okay." if that variable is set.

I think this does not distrub any operations, as it only modifies the error message. If there is a target called "me", make should behave usual.

If you would be willing to include such a "feature", I’ll provide a patch.

Thanks,
Joachim

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #16091:  make_me_some_tea.patch added by jamesc (620B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jamesc (Updated the item)
  • -email is unavailable- added by nomeata (Posted a comment)
  • -email is unavailable- added by fangism (Posted a comment)
  • -email is unavailable- added by davek (Posted a comment)
  • -email is unavailable- added by davek (Voted in favor of this item)
  • -email is unavailable- added by nomeata (Hadn’t created an account when I submitted the bug.)
  •  

    There is 1 vote so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2008-07-10 jamesc Attached File- Added make_me_some_tea.patch, #16091
    2008-05-17 davek Carbon-Copy- Added davek
    2008-05-17 nomeata Carbon-Copy- Added nomeata

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code