Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home
« Groups Home
Help needed: fedora package update
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  10 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Olivier Bilodeau  
View profile  
 More options Apr 18 2012, 2:29 pm
From: Olivier Bilodeau <obilod...@inverse.ca>
Date: Wed, 18 Apr 2012 14:29:04 -0400
Local: Wed, Apr 18 2012 2:29 pm
Subject: Help needed: fedora package update

Hi,

I would like to update the fedora package for asciidoc as our software's
documentation requires an asciidoc >= 8.6.2 to be built and fedora still
ships with 8.4.5 (even in rawhide).

One of the patches no longer applies cleanly and I wanted to see if it
was implemented upstream or not. It's called datadir (see attached
asciidoc-8.4.5-datadir.patch). And I couldn't figure it out..

Fails with:
1 out of 2 hunks FAILED -- saving rejects to file asciidoc.py.rej
4 out of 4 hunks FAILED -- saving rejects to file Makefile.in.rej

The other one has been integrated[1] so I'm droppping it.

Thanks,

[1]
http://groups.google.com/group/asciidoc/browse_frm/thread/ea3a8ea399a...
--
Olivier Bilodeau
obilod...@inverse.ca  ::  +1.514.447.4918 *115  ::  www.inverse.ca
Inverse inc. :: Leaders behind SOGo (www.sogo.nu) and PacketFence
(www.packetfence.org)

  asciidoc-8.4.5-datadir.patch
2K Download

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Lex Trotman  
View profile  
 More options Apr 18 2012, 8:29 pm
From: Lex Trotman <ele...@gmail.com>
Date: Thu, 19 Apr 2012 10:29:37 +1000
Local: Wed, Apr 18 2012 8:29 pm
Subject: Re: Help needed: fedora package update
On 19 April 2012 04:29, Olivier Bilodeau <obilod...@inverse.ca> wrote:

> Hi,

> I would like to update the fedora package for asciidoc as our software's
> documentation requires an asciidoc >= 8.6.2 to be built and fedora still
> ships with 8.4.5 (even in rawhide).

> One of the patches no longer applies cleanly and I wanted to see if it
> was implemented upstream or not. It's called datadir (see attached
> asciidoc-8.4.5-datadir.patch). And I couldn't figure it out..

Hi,

No its not integrated, but I can't figure out what it is trying to achieve?

Note, the handling of all plugins was unified in 8.6.6 for backends,
filters and themes and that is probably why it doesn't apply.  With
this change the patch might not be required any more or it will need
to be completely re-written depending on what it is trying to do.

Cheers
Lex


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Olivier Bilodeau  
View profile  
 More options May 1 2012, 3:54 pm
From: Olivier Bilodeau <obilod...@inverse.ca>
Date: Tue, 01 May 2012 15:54:39 -0400
Local: Tues, May 1 2012 3:54 pm
Subject: Re: Help needed: fedora package update
Hi Lex,

Thanks for the reply.

It seems to put most of asciidoc's files under /usr/share/asciidoc/
instead of /etc/asciidoc/ in order to align with the Filesystem
Hierarchy Standard (FHS)[1]. Which says:

"The /etc hierarchy contains configuration files. A "configuration file"
is a local file used to control the operation of a program; it must be
static and cannot be an executable binary."

Is it something that asciidoc supports through it's Makefile now?

In any case, I'll look at what the debian package does and I'll update
the spec accordingly.

[1]: http://www.pathname.com/fhs/pub/fhs-2.3.html
--
Olivier Bilodeau
obilod...@inverse.ca  ::  +1.514.447.4918 *115  ::  www.inverse.ca
Inverse inc. :: Leaders behind SOGo (www.sogo.nu) and PacketFence
(www.packetfence.org)


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Lex Trotman  
View profile  
 More options May 1 2012, 8:48 pm
From: Lex Trotman <ele...@gmail.com>
Date: Wed, 2 May 2012 10:48:16 +1000
Local: Tues, May 1 2012 8:48 pm
Subject: Re: Help needed: fedora package update
On 2 May 2012 05:54, Olivier Bilodeau <obilod...@inverse.ca> wrote:

Hi,

Ok, I see, it is high time distros stopped just dumping stuff in one
directory, but that gripe is OT for this list :)

On my machine it looks like debian configured it with prefix=/usr
datadir=/usr/share/asciidoc sysconfdir=/etc/asciidoc

But they must also have changed progsymlink to rename and not link the
"executables" asciidoc and a2x since they are python files without an
extension (bad Karma there too :(

Good luck.

Cheers
Lex


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Lex Trotman  
View profile  
 More options May 1 2012, 9:25 pm
From: Lex Trotman <ele...@gmail.com>
Date: Wed, 2 May 2012 11:25:28 +1000
Local: Tues, May 1 2012 9:25 pm
Subject: Re: Help needed: fedora package update
[...]

> But they must also have changed progsymlink to rename and not link the
> "executables" asciidoc and a2x since they are python files without an
> extension (bad Karma there too :(

Hi again,

Told you it was bad Karma, this breaks Python import which requires
.py extension.

Since asciidocapi imports asciidoc it has to load the source manually
instead of letting the Python process work, and that disables
pre-compiled caching, meaning it has to compile the source each time.

Since that mess exists in asciidocapi you can get away with it if you
*MUST*, but it would be better if the normal asciidoc link method was
used so caching worked.

Or of course if you can suggest some alternative method that meets
both requirements and is suitably backward compatible that would be
even better :)

Cheers
Lex


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Olivier Bilodeau  
View profile  
 More options May 4 2012, 8:57 pm
From: Olivier Bilodeau <obilod...@inverse.ca>
Date: Fri, 04 May 2012 20:57:16 -0400
Local: Fri, May 4 2012 8:57 pm
Subject: Re: Help needed: fedora package update

Hi Lex,

> Told you it was bad Karma, this breaks Python import which requires
> .py extension.

> Since asciidocapi imports asciidoc it has to load the source manually
> instead of letting the Python process work, and that disables
> pre-compiled caching, meaning it has to compile the source each time.

> Since that mess exists in asciidocapi you can get away with it if you
> *MUST*, but it would be better if the normal asciidoc link method was
> used so caching worked.

> Or of course if you can suggest some alternative method that meets
> both requirements and is suitably backward compatible that would be
> even better :)

I found a patch in the repoforge repository that applies cleanly and
meets the FHS requirement. I've updated the SPEC[1] and it also installs
the vim ftdetect and plugin. All stolen from Dag's work at repoforge. ;)

I attached the patch. Is this a "bad karma" patch? For us asciidoc has
been performing up to expectation with this and anyways compared to fop
I'm pretty sure it'll always be the faster part of the document
rendering process.

I'll try to get in touch with Fedora's asciidoc package maintainers and
have them accept this. If you have any comment let me know.

Sorry if my help is rather hackish but python and distro-grade packaging
is not my fort� but we rely on a modern asciidoc and I would rather
prefer to have it upstream in Fedora than downstream in our repo or from
source.

Cheers!

[1]: https://github.com/inverse-inc/asciidoc.spec
--
Olivier Bilodeau
obilod...@inverse.ca  ::  +1.514.447.4918 *115  ::  www.inverse.ca
Inverse inc. :: Leaders behind SOGo (www.sogo.nu) and PacketFence
(www.packetfence.org)

  asciidoc-8.6.1-datadir.patch
2K Download

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Olivier Bilodeau  
View profile  
 More options May 7 2012, 12:45 pm
From: Olivier Bilodeau <obilod...@inverse.ca>
Date: Mon, 07 May 2012 12:45:55 -0400
Local: Mon, May 7 2012 12:45 pm
Subject: Re: Help needed: fedora package update

> What license is Repoforge?  I couldn't find one, but that probably
> means it beneath my nose :)

I've taken one patch and according to a Repoforge admin in #repoforge,
it's fine:

We went on a little bit and, of course, he told me he is not a lawyer.
Due to size of the said "borrowed work" and the positive reaction of
someone embodying that community, I think it'll be an acceptable
contribution for Fedora.

I contacted the maintainer. He gave me two other things to do:

- vim plugin should be separately [sub]packaged
- build the git doc with it and it shall still be pretty

I'll get to tasks, eventually, and let you guys know.

Cheers!
--
Olivier Bilodeau
obilod...@inverse.ca  ::  +1.514.447.4918 *115  ::  www.inverse.ca
Inverse inc. :: Leaders behind SOGo (www.sogo.nu) and PacketFence
(www.packetfence.org)


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Dag Wieers  
View profile  
 More options May 9 2012, 4:42 am
From: Dag Wieers <d...@wieers.com>
Date: Wed, 9 May 2012 10:42:31 +0200 (CEST)
Local: Wed, May 9 2012 4:42 am
Subject: Re: Help needed: fedora package update

On Mon, 7 May 2012, Olivier Bilodeau wrote:
> I contacted the maintainer. He gave me two other things to do:

> - vim plugin should be separately [sub]packaged
> - build the git doc with it and it shall still be pretty

> I'll get to tasks, eventually, and let you guys know.

I'll give you my view of sub-packages on this. I think Fedora is making
the mistake to create too many sub-packages and loosing control over what
makes packages useful.

I don't see the point in making a separate vim sub-package, given that the
vim syntax file is pretty small, a sub-package does not offer anything
subtantially, a subpackage increases the chance that someone will not
install it and does not know it exists which is bad for the repository and
the project (in this case vim and/or asciidoc) and most of all, there is
no real downside to shipping the plugin even when vim is not installed.

So my opinion is that these sub-package rules is a form of bureaucratic
autism, where any introduced rules are more important than the case at
hand and defy common sense.

Of course, there are cases where they can make sense.

--
-- dag wieers, d...@wieers.com, http://dag.wieers.com/
-- dagit linux solutions, i...@dagit.net, http://dagit.net/

[Any errors in spelling, tact or fact are transmission errors]


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Olivier Bilodeau  
View profile  
 More options May 9 2012, 8:57 am
From: Olivier Bilodeau <obilod...@inverse.ca>
Date: Wed, 09 May 2012 08:57:44 -0400
Local: Wed, May 9 2012 8:57 am
Subject: Re: Help needed: fedora package update
On 05/09/2012 04:42 AM, Dag Wieers wrote:

I see your point. I'll ask the maintainer.

However I just realized that most vim ftplugin are provided by
vim-commons (it provides 175 of them) and not individually provided by
git, dpkg, mplayer or xfree86 (as a few examples).

So maybe that's the route it should take?

--
Olivier Bilodeau
obilod...@inverse.ca  ::  +1.514.447.4918 *115  ::  www.inverse.ca
Inverse inc. :: Leaders behind SOGo (www.sogo.nu) and PacketFence
(www.packetfence.org)


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Dag Wieers  
View profile  
 More options May 9 2012, 9:26 am
From: Dag Wieers <d...@wieers.com>
Date: Wed, 9 May 2012 15:26:29 +0200 (CEST)
Local: Wed, May 9 2012 9:26 am
Subject: Re: Help needed: fedora package update

On Wed, 9 May 2012, Olivier Bilodeau wrote:
> However I just realized that most vim ftplugin are provided by
> vim-commons (it provides 175 of them) and not individually provided by
> git, dpkg, mplayer or xfree86 (as a few examples).

> So maybe that's the route it should take?

It depends. If it is shipped as part of asciidoc it is easier to
maintain/update by Stuart, which may be harder when it ships with vim.

AFAIK all of the syntax-files in vim-common are maintained as part of vim.
So the real question is if Stuart is interested in this.

PS This is not a packager's decision IMO. Otherwise it becomes a
maintenance nightmare.

--
-- dag wieers, d...@wieers.com, http://dag.wieers.com/
-- dagit linux solutions, i...@dagit.net, http://dagit.net/

[Any errors in spelling, tact or fact are transmission errors]


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »