Checks
Context |
Check |
Description |
linaro-tcwg-bot/tcwg_gcc_build--master-arm |
success
|
Build passed
|
linaro-tcwg-bot/tcwg_gcc_build--master-aarch64 |
success
|
Build passed
|
linaro-tcwg-bot/tcwg_gcc_check--master-aarch64 |
success
|
Test passed
|
linaro-tcwg-bot/tcwg_gcc_check--master-arm |
success
|
Test passed
|
Commit Message
Fix typos in comments.
Signed-off-by: Andrew Kreimer <algonell@gmail.com>
---
ltmain.sh | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Comments
Hi!
Thanks for the patch.
Andrew Kreimer <algonell@gmail.com> writes:
> Fix typos in comments.
I'm not sure we should further diverge our local libtool copy compared
to upstream. Best check whether the same errors exist in upstream
libtool ( https://savannah.gnu.org/projects/libtool/ ) and fix them
there, I think.
On Wed, Sep 18, 2024 at 09:43:03PM +0200, Arsen Arsenović wrote:
> Hi!
>
> Thanks for the patch.
>
> Andrew Kreimer <algonell@gmail.com> writes:
>
> > Fix typos in comments.
>
> I'm not sure we should further diverge our local libtool copy compared
> to upstream. Best check whether the same errors exist in upstream
> libtool ( https://savannah.gnu.org/projects/libtool/ ) and fix them
> there, I think.
> --
> Arsen Arsenović
OK.
@@ -177,7 +177,7 @@ basename="s,^.*/,,"
# to NONDIR_REPLACEMENT.
# value returned in "$func_dirname_result"
# basename: Compute filename of FILE.
-# value retuned in "$func_basename_result"
+# value returned in "$func_basename_result"
# Implementation must be kept synchronized with func_dirname
# and func_basename. For efficiency, we do not delegate to
# those functions but instead duplicate the functionality here.
@@ -487,7 +487,7 @@ func_mkdir_p ()
# While some portion of DIR does not yet exist...
while test ! -d "$my_directory_path"; do
# ...make a list in topmost first order. Use a colon delimited
- # list incase some portion of path contains whitespace.
+ # list in case some portion of path contains whitespace.
my_dir_list="$my_directory_path:$my_dir_list"
# If the last portion added has no slash in it, the list is done
@@ -3674,7 +3674,7 @@ EOF
{
/* however, if there is an option in the LTWRAPPER_OPTION_PREFIX
namespace, but it is not one of the ones we know about and
- have already dealt with, above (inluding dump-script), then
+ have already dealt with, above (including dump-script), then
report an error. Otherwise, targets might begin to believe
they are allowed to use options in the LTWRAPPER_OPTION_PREFIX
namespace. The first time any user complains about this, we'll