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