Merged repo for protocol headers? Why are they split?

[Daniel Martin]

Am 07.12.2017 17:18 schrieb "Emil Velikov" <emil.l.velikov at gmail.com>:

On 6 December 2017 at 12:37, Daniel Martin <consume.noise at gmail.com> wrote:
> Hi all,
>
> if anyone would like to have a look, I've pushed my current work on
> the merged proto repo here:
>     https://github.com/bartsch/xorg-proto2k/
> It's generated as is with:
>     https://github.com/bartsch/proto2k-generator/
>
> I used git-filter-branch to:
> - move files to specific directories and
> - prefix commit subjects with the proto name.
>
> Lots of files have been moved to tmp/, which was removed at the end.
> See commit "Remove temporary files (tmp/)".
>
> Atm, it just installs:
> - headers (+handling Xfuncproto.h and Xpoll.h) and
> - pkg-configs for each proto (haven't found any notable diff'erences
> to my distro pkg-configs)
>
> Now:
> - What should I do with the {AUTHORS,COPYING}.$proto files?
> - Should I remove some protos (i.e. trapproto)? The current list is:
>     https://github.com/bartsch/xorg-proto2k/commit/
8b06f11ffff1f99c2f66a9327ef669ee17c530e7
> - Anything else I should change?
>
> Next:
> - I'm working on generating the specs.
>
Silly question: why are you moving things around?


I did that to reflect the installation hierarchy (at least for the headers)
and filter out files (i.e. autofoo and Makefiles) we don't need later.

Perhaps I misread something, but it seemed that devs prefer to have
the different repos in separate folders.
Aka something like
for repo in $repos; do
    mv .../old/to/old/repo/$repo .../new/to/newrepo/$repo
done


I had the impression that this was the preferred way with git-subtree as it
doesn't allow us anything else without conflicts.

The repo you're prepared looks quite messy IMHO.


Can you be more specific, please? The mixed header files?

I can and will change stuff. I just need someone saying: We would like to
have it like ...
Though, not this week as I've a full time RL task atm..

Cheers,
    Daniel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.x.org/archives/xorg-devel/attachments/20171213/a5709207/attachment-0001.html>