Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 1 | How to Add a New Applet to BusyBox |
| 2 | ================================== |
| 3 | |
| 4 | This document details the steps you must take to add a new applet to BusyBox. |
| 5 | |
| 6 | Credits: |
| 7 | Matt Kraai - initial writeup |
| 8 | Mark Whitley - the remix |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 9 | Thomas Lundquist - Trying to keep it updated. |
| 10 | |
Dan Fandrich | 9669363 | 2010-08-10 23:45:27 -0700 | [diff] [blame] | 11 | When doing this you should consider using the latest git HEAD. |
Denys Vlasenko | 5370bfb | 2009-09-06 02:58:59 +0200 | [diff] [blame] | 12 | This is a good thing if you plan to getting it committed into mainline. |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 13 | |
| 14 | Initial Write |
| 15 | ------------- |
| 16 | |
Mike Frysinger | 9754b91 | 2005-09-02 23:06:30 +0000 | [diff] [blame] | 17 | First, write your applet. Be sure to include copyright information at the top, |
| 18 | such as who you stole the code from and so forth. Also include the mini-GPL |
| 19 | boilerplate. Be sure to name the main function <applet>_main instead of main. |
| 20 | And be sure to put it in <applet>.c. Usage does not have to be taken care of by |
| 21 | your applet. |
Denys Vlasenko | 02788ac | 2010-10-28 20:45:37 +0200 | [diff] [blame] | 22 | Make sure to #include "libbb.h" as the first include file in your applet. |
Glenn L McGrath | 9c91e41 | 2003-10-01 11:33:46 +0000 | [diff] [blame] | 23 | |
| 24 | For a new applet mu, here is the code that would go in mu.c: |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 25 | |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 26 | (busybox.h already includes most usual header files. You do not need |
| 27 | #include <stdio.h> etc...) |
| 28 | |
| 29 | |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 30 | ----begin example code------ |
| 31 | |
| 32 | /* vi: set sw=4 ts=4: */ |
| 33 | /* |
| 34 | * Mini mu implementation for busybox |
| 35 | * |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 36 | * Copyright (C) [YEAR] by [YOUR NAME] <YOUR EMAIL> |
| 37 | * |
Denys Vlasenko | 0ef64bd | 2010-08-16 20:14:46 +0200 | [diff] [blame] | 38 | * Licensed under GPLv2, see file LICENSE in this source tree. |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 39 | */ |
| 40 | |
Denis Vlasenko | b6adbf1 | 2007-05-26 19:00:18 +0000 | [diff] [blame] | 41 | #include "libbb.h" |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 42 | #include "other.h" |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 43 | |
Denis Vlasenko | 9b49a5e | 2007-10-11 10:05:36 +0000 | [diff] [blame] | 44 | int mu_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 45 | int mu_main(int argc, char **argv) |
| 46 | { |
| 47 | int fd; |
Denis Vlasenko | 360362d | 2007-11-04 04:46:46 +0000 | [diff] [blame] | 48 | ssize_t n; |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 49 | char mu; |
| 50 | |
Bernhard Reutner-Fischer | 8c1eda5 | 2006-08-28 23:39:36 +0000 | [diff] [blame] | 51 | fd = xopen("/dev/random", O_RDONLY); |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 52 | |
| 53 | if ((n = safe_read(fd, &mu, 1)) < 1) |
Mike Frysinger | 0ea3a6f | 2005-04-23 01:43:45 +0000 | [diff] [blame] | 54 | bb_perror_msg_and_die("/dev/random"); |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 55 | |
| 56 | return mu; |
| 57 | } |
| 58 | |
| 59 | ----end example code------ |
| 60 | |
Mark Whitley | 8a6b619 | 2000-12-19 17:54:38 +0000 | [diff] [blame] | 61 | |
| 62 | Coding Style |
| 63 | ------------ |
| 64 | |
| 65 | Before you submit your applet for inclusion in BusyBox, (or better yet, before |
| 66 | you _write_ your applet) please read through the style guide in the docs |
| 67 | directory and make your program compliant. |
| 68 | |
| 69 | |
Glenn L McGrath | 2e6d3cf | 2001-06-24 12:36:54 +0000 | [diff] [blame] | 70 | Some Words on libbb |
| 71 | ------------------- |
Mark Whitley | 8a6b619 | 2000-12-19 17:54:38 +0000 | [diff] [blame] | 72 | |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 73 | As you are writing your applet, please be aware of the body of pre-existing |
Glenn L McGrath | 2e6d3cf | 2001-06-24 12:36:54 +0000 | [diff] [blame] | 74 | useful functions in libbb. Use these instead of reinventing the wheel. |
Mark Whitley | 8a6b619 | 2000-12-19 17:54:38 +0000 | [diff] [blame] | 75 | |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 76 | Additionally, if you have any useful, general-purpose functions in your |
Mike Frysinger | 9754b91 | 2005-09-02 23:06:30 +0000 | [diff] [blame] | 77 | applet that could be useful in other applets, consider putting them in libbb. |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 78 | |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 79 | And it may be possible that some of the other applets uses functions you |
| 80 | could use. If so, you have to rip the function out of the applet and make |
| 81 | a libbb function out of it. |
| 82 | |
| 83 | Adding a libbb function: |
| 84 | ------------------------ |
| 85 | |
| 86 | Make a new file named <function_name>.c |
| 87 | |
| 88 | ----start example code------ |
| 89 | |
| 90 | #include "libbb.h" |
| 91 | #include "other.h" |
| 92 | |
| 93 | int function(char *a) |
| 94 | { |
| 95 | return *a; |
| 96 | } |
| 97 | |
| 98 | ----end example code------ |
| 99 | |
Denis Vlasenko | 4b924f3 | 2007-05-30 00:29:55 +0000 | [diff] [blame] | 100 | Add <function_name>.o in the right alphabetically sorted place |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 101 | in libbb/Kbuild.src. You should look at the conditional part of |
| 102 | libbb/Kbuild.src as well. |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 103 | |
Denis Vlasenko | 4b924f3 | 2007-05-30 00:29:55 +0000 | [diff] [blame] | 104 | You should also try to find a suitable place in include/libbb.h for |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 105 | the function declaration. If not, add it somewhere anyway, with or without |
| 106 | ifdefs to include or not. |
| 107 | |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 108 | You can look at libbb/Config.src and try to find out if the function is |
Denys Vlasenko | 5370bfb | 2009-09-06 02:58:59 +0200 | [diff] [blame] | 109 | tunable and add it there if it is. |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 110 | |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 111 | |
Matt Kraai | 3b1cbd7 | 2002-03-18 16:03:00 +0000 | [diff] [blame] | 112 | Placement / Directory |
| 113 | --------------------- |
| 114 | |
| 115 | Find the appropriate directory for your new applet. |
| 116 | |
Glenn L McGrath | 9c91e41 | 2003-10-01 11:33:46 +0000 | [diff] [blame] | 117 | Make sure you find the appropriate places in the files, the applets are |
| 118 | sorted alphabetically. |
| 119 | |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 120 | Add the applet to Kbuild.src in the chosen directory: |
Matt Kraai | 3b1cbd7 | 2002-03-18 16:03:00 +0000 | [diff] [blame] | 121 | |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 122 | lib-$(CONFIG_MU) += mu.o |
Matt Kraai | 3b1cbd7 | 2002-03-18 16:03:00 +0000 | [diff] [blame] | 123 | |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 124 | Add the applet to Config.src in the chosen directory: |
Matt Kraai | 3b1cbd7 | 2002-03-18 16:03:00 +0000 | [diff] [blame] | 125 | |
Denis Vlasenko | 360362d | 2007-11-04 04:46:46 +0000 | [diff] [blame] | 126 | config MU |
Glenn L McGrath | 9c91e41 | 2003-10-01 11:33:46 +0000 | [diff] [blame] | 127 | bool "MU" |
| 128 | default n |
| 129 | help |
| 130 | Returns an indeterminate value. |
Matt Kraai | 3b1cbd7 | 2002-03-18 16:03:00 +0000 | [diff] [blame] | 131 | |
| 132 | |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 133 | Usage String(s) |
| 134 | --------------- |
| 135 | |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 136 | Next, add usage information for you applet to include/usage.src.h. |
Matt Kraai | 3b1cbd7 | 2002-03-18 16:03:00 +0000 | [diff] [blame] | 137 | This should look like the following: |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 138 | |
Glenn L McGrath | 2e6d3cf | 2001-06-24 12:36:54 +0000 | [diff] [blame] | 139 | #define mu_trivial_usage \ |
| 140 | "-[abcde] FILES" |
| 141 | #define mu_full_usage \ |
| 142 | "Returns an indeterminate value.\n\n" \ |
| 143 | "Options:\n" \ |
| 144 | "\t-a\t\tfirst function\n" \ |
| 145 | "\t-b\t\tsecond function\n" \ |
Mike Frysinger | 9754b91 | 2005-09-02 23:06:30 +0000 | [diff] [blame] | 146 | ... |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 147 | |
| 148 | If your program supports flags, the flags should be mentioned on the first |
Glenn L McGrath | 2e6d3cf | 2001-06-24 12:36:54 +0000 | [diff] [blame] | 149 | line (-[abcde]) and a detailed description of each flag should go in the |
| 150 | mu_full_usage section, one flag per line. (Numerous examples of this |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 151 | currently exist in usage.src.h.) |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 152 | |
| 153 | |
| 154 | Header Files |
| 155 | ------------ |
| 156 | |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 157 | Next, add an entry to include/applets.src.h. Be *sure* to keep the list |
Eric Andersen | c7bda1c | 2004-03-15 08:29:22 +0000 | [diff] [blame] | 158 | in alphabetical order, or else it will break the binary-search lookup |
Matt Kraai | 3b1cbd7 | 2002-03-18 16:03:00 +0000 | [diff] [blame] | 159 | algorithm in busybox.c and the Gods of BusyBox smite you. Yea, verily: |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 160 | |
Holger Blasum | b27d62a | 2010-10-28 20:37:05 +0200 | [diff] [blame] | 161 | Be sure to read the top of applets.src.h before adding your applet. |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 162 | |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 163 | /* all programs above here are alphabetically "less than" 'mu' */ |
Denys Vlasenko | b9f2d9f | 2011-01-18 13:58:01 +0100 | [diff] [blame] | 164 | IF_MU(APPLET(mu, BB_DIR_USR_BIN, BB_SUID_DROP)) |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 165 | /* all programs below here are alphabetically "greater than" 'mu' */ |
| 166 | |
| 167 | |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 168 | The Grand Announcement |
| 169 | ---------------------- |
| 170 | |
Dan Fandrich | 9669363 | 2010-08-10 23:45:27 -0700 | [diff] [blame] | 171 | Then create a diff by adding the new files to git (remember your libbb files) |
| 172 | git add <where you put it>/mu.c |
Denis Vlasenko | 4b924f3 | 2007-05-30 00:29:55 +0000 | [diff] [blame] | 173 | eventually also: |
Dan Fandrich | 9669363 | 2010-08-10 23:45:27 -0700 | [diff] [blame] | 174 | git add libbb/function.c |
Denis Vlasenko | 7e84e53 | 2007-05-08 17:52:17 +0000 | [diff] [blame] | 175 | then |
Dan Fandrich | 9669363 | 2010-08-10 23:45:27 -0700 | [diff] [blame] | 176 | git commit |
| 177 | git format-patch HEAD^ |
Glenn L McGrath | 9c91e41 | 2003-10-01 11:33:46 +0000 | [diff] [blame] | 178 | and send it to the mailing list: |
Mike Frysinger | 9754b91 | 2005-09-02 23:06:30 +0000 | [diff] [blame] | 179 | busybox@busybox.net |
| 180 | http://busybox.net/mailman/listinfo/busybox |
Mark Whitley | eac2636 | 2000-12-19 17:35:24 +0000 | [diff] [blame] | 181 | |
Glenn L McGrath | 9c91e41 | 2003-10-01 11:33:46 +0000 | [diff] [blame] | 182 | Sending patches as attachments is preferred, but not required. |