# Extension building problems

- Thread: https://externals.io/message/3890
- Original archive: https://news-web.php.net/php.internals/3890
- Started by Brian J. France on 2003-08-13T22:08:00+00:00
- Messages: 4
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #3890 — Brian J. France — 2003-08-13T22:08:00+00:00

URL: https://externals.io/message/3890#3890

I am having problem building extension with long names.  As an example say my 
extension name is foooo_naaaaame_baaaaar and 
FOOOO_NAAAAAME_BAAAAAR_SHARED_LIBADD is used to link libraries.

The problem is when the Makefile is generated I get 
FOOOO_NAAAAAME_BAAAAAR_SHARED_LIBADD at the top of the Makefile, but 
FOOOONAAAAAMEBAAAAAR_SHARED_LIBADD at the bottom for the link line.

If I change line 1224 of acinclude.m4 from:

$3/$1.la: \$($2) \$(translit($1,a-z-,A-Z_)_SHARED_DEPENDENCIES)

to: 

$3/$1.la: \$($2) \$(translit($1,a-z_-,A-Z__)_SHARED_DEPENDENCIES)

it works fine.  What else is strange is that if I shorten the name it works 
with out the change (foooo_baaaaar works).

Did something change recently to cause this? If nobody sees a problem with the 
change I am going to check it in.

Thanks,

Brian

---

## #3891 — Sascha Schumann — 2003-08-13T22:17:11+00:00

URL: https://externals.io/message/3890#3891 · In reply to: #3890

> Did something change recently to cause this? If nobody sees a
> problem with the change I am going to check it in.

    There are some translit instances which should be aligned
    with common usage.  Please submit a patch for review.

    - Sascha

---

## #3893 — Brian J. France — 2003-08-13T22:31:36+00:00

URL: https://externals.io/message/3890#3893 · In reply to: #3891

Here is a patch, it is actually two lines that need changed.

Thanks,

Brian


> > […]
> 
>     There are some translit instances which should be aligned
>     with common usage.  Please submit a patch for review.
> 
>     - Sascha

---

## #3894 — Sascha Schumann — 2003-08-13T22:35:33+00:00

URL: https://externals.io/message/3890#3894 · In reply to: #3893

> Here is a patch, it is actually two lines that need changed.

    Ok to commit to all relevant branches.

    - Sascha